diff options
| author | Schrodinger ZHU Yifan <yifanzhu@rochester.edu> | 2024-11-26 15:15:58 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-11-26 15:15:58 -0500 |
| commit | c71418574f1bb9e4678428901775c8b633cded09 (patch) | |
| tree | fa4072933e0b5f822bd4998b2cb5cb60c7ce2133 /libc/src/string | |
| parent | c9562b181cafafba104cccd7334ace1fd6aee5ac (diff) | |
[libc] suppress more clang-cl warnings (#117718)
- migrate more `-O3` to `${libc_opt_high_flag}`
- workaround a issue with `LLP64` in test. The overflow testing is
guarded by a constexpr but the literal overflow itself will still
trigger warnings.
Notice that for math smoke test, for some reasons, the
`${libc_opt_high_flag}` will be passed into `lld-link` which confuses
the linker so there are still some warnings leftover there. I can
investigate more when I have time.
Diffstat (limited to 'libc/src/string')
| -rw-r--r-- | libc/src/string/CMakeLists.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libc/src/string/CMakeLists.txt b/libc/src/string/CMakeLists.txt index d6e300754d4f..8fe1226dd6a7 100644 --- a/libc/src/string/CMakeLists.txt +++ b/libc/src/string/CMakeLists.txt @@ -506,7 +506,7 @@ function(add_implementation name impl_name) SRCS ${ADD_IMPL_SRCS} HDRS ${ADD_IMPL_HDRS} DEPENDS ${ADD_IMPL_DEPENDS} - COMPILE_OPTIONS -O3 ${ADD_IMPL_COMPILE_OPTIONS} + COMPILE_OPTIONS ${libc_opt_high_flag} ${ADD_IMPL_COMPILE_OPTIONS} ) get_fq_target_name(${impl_name} fq_target_name) set_target_properties(${fq_target_name} PROPERTIES REQUIRE_CPU_FEATURES "${ADD_IMPL_REQUIRE}") |