Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: check the combination of Sanitizers #2408

Merged
merged 1 commit into from
Sep 11, 2024

Commits on Sep 2, 2024

  1. build: check the combination of Sanitizers

    we are going to add a build with ThreadSanitizer enabled, but
    ThreadSanitizer cannot be used along with AddressSanitizer. the
    existing sanitizer detection checks each sanitizer separately, so
    even if the sanitizers combination does not work, the detection
    still passes.
    
    also, we hardwire two sanitizers in `FindSanitizers.cmake`, this
    is not extensible to the use case where we only need to selectively
    find a certain (combination) of sanitizers.
    
    in order to address these problems, in this change
    
    * find specified component(s) in FindSanitizers.cmake, to prepare for
      the change which selectively specifies a subset of sanitizers
    * check if the compiler supports the combination of compile options
      required by all specified sanitizers
    
    Signed-off-by: Kefu Chai <[email protected]>
    tchaikov committed Sep 2, 2024
    Configuration menu
    Copy the full SHA
    d6dc2ac View commit details
    Browse the repository at this point in the history