Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 3, 2025

ASAN_RUNTIME_DEP was referenced in the EXTRA_RUNTIME_DEPS parameter but never defined. ASAN DLL handling is already covered by the install command in root CMakeLists.txt.

  • Removed $<$<CONFIG:Debug>:;${ASAN_RUNTIME_DEP}> generator expression from the POST_BUILD custom command

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove ASAN_RUNTIME_DEP from mimallocator support PR Remove undefined ASAN_RUNTIME_DEP from runtime deps Dec 3, 2025
Copilot AI requested a review from asn5d December 3, 2025 23:26
@asn5d asn5d marked this pull request as ready for review December 4, 2025 17:33
@asn5d asn5d merged commit 99c9cf7 into feature/mimalloc Dec 4, 2025
@asn5d asn5d deleted the copilot/sub-pr-6-again branch December 4, 2025 17:33
asn5d added a commit that referenced this pull request Dec 4, 2025
* Add optional mimalloc allocator support for Release builds.

* Update RoiManager/CMakeLists.txt to check mimalloc target exists before linking.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update PluginManager/CMakeLists.txt to check mimalloc target exists before linking.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update CMakeLists.txt to show message that mimalloc will be linked for Release only.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Added link option /INCLUDE:mi_version in cvutil/CMakeLists.txt  only for MSVC

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Updated CMakeLists.txt to include platform specific import locations.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix generator expression in ExternalProject_Add for mimalloc (#8)

* Initial plan

* Replace $<CONFIG> with ${CMAKE_BUILD_TYPE} in ExternalProject_Add

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

* Use hardcoded Release build type for mimalloc ExternalProject

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

* Remove undefined ASAN_RUNTIME_DEP from runtime deps (#9)

* Initial plan

* Remove undefined ASAN_RUNTIME_DEP from EXTRA_RUNTIME_DEPS

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

* Remove undefined ASAN_RUNTIME_DEP from runtime deps (#10)

* Initial plan

* Remove undefined ASAN_RUNTIME_DEP from EXTRA_RUNTIME_DEPS expression

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

* Use $<IF> syntax for EXTRA_RUNTIME_DEPS generator expression

Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: asn5d <16312669+asn5d@users.noreply.github.com>

---------

Co-authored-by: Seethepalli <7uz@ornl.gov>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants