Skip to content

build: fix memory manager compile options for CMake#13347

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
Jasinsky:sim_ostest_mm_cmake_fix
Sep 10, 2024
Merged

build: fix memory manager compile options for CMake#13347
xiaoxiang781216 merged 1 commit into
apache:masterfrom
Jasinsky:sim_ostest_mm_cmake_fix

Conversation

@Jasinsky

@Jasinsky Jasinsky commented Sep 9, 2024

Copy link
Copy Markdown
Contributor

Summary

This enables CMake based build for sim:ostest. Flags which are set by set_source_files_properties are not reflected in final build. Changing these to target_compile_options makes CMake build runnable.

Impact

CMake based build for sim:ostest now runs.

Testing

Local build and playing around sim:ostest

This enables CMake based build for sim:ostest. Flags which
are set by set_source_files_properties are not reflected in
final build. Changing these to target_compile_options makes
CMake build runnable.

Signed-off-by: Daniel Jasinski <jasinskidaniel95szcz@gmail.com>
@Jasinsky

Jasinsky commented Sep 9, 2024

Copy link
Copy Markdown
Contributor Author

FYI @acassis @xiaoxiang781216

@Jasinsky

Jasinsky commented Sep 9, 2024

Copy link
Copy Markdown
Contributor Author

#13302 will be closed by this issue.

Comment thread mm/kasan/CMakeLists.txt
@xiaoxiang781216 xiaoxiang781216 merged commit 7dbb887 into apache:master Sep 10, 2024
@Jasinsky Jasinsky deleted the sim_ostest_mm_cmake_fix branch January 26, 2025 18:56
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.

[BUG] sim:ostest receives SIGSEGV in kasan_mem_to_shadow when build with CMake

3 participants