[all-commits] [llvm/llvm-project] 9bd1ff: [CMake][ASAN] Add support for ADDRESS_SANITIZER_BU...

[Christian Ulmann via All-commits]

  Branch: refs/heads/users/dinistro/asan-compile-option
  Home:   https://github.com/llvm/llvm-project
  Commit: 9bd1fffae741322f61cc408623fef2a507b1bd0f
      https://github.com/llvm/llvm-project/commit/9bd1fffae741322f61cc408623fef2a507b1bd0f
  Author: Christian Ulmann <christian.ulmann at nextsilicon.com>
  Date:   2024-08-04 (Sun, 04 Aug 2024)

  Changed paths:
    M llvm/cmake/modules/HandleLLVMOptions.cmake
    M llvm/include/llvm/Support/Compiler.h

  Log Message:
  -----------
  [CMake][ASAN] Add support for ADDRESS_SANITIZER_BUILD compile option

This commit introduces the `LLVM_ADDRESS_SANITIZER_BUILD` compile option
that is set to indicate if LLVM has been built with ASAN enabled or not.
This is a necessary addition to properly control the declarations of
`__asan_*` helpers in header files. Previously, some of these
declarations relied on `__has_feature(address_sanitizer)`, which can
lead to differences between in-tree compilation units and down-stream
usages of the same headers.

The reproducer of this issue involved templates, which then lead to
violation of the ODR due to having two different instances of the same
template.


  Commit: cf02983f4375a32f35acfcdda98201f38e8776fe
      https://github.com/llvm/llvm-project/commit/cf02983f4375a32f35acfcdda98201f38e8776fe
  Author: Christian Ulmann <christian.ulmann at nextsilicon.com>
  Date:   2024-08-04 (Sun, 04 Aug 2024)

  Changed paths:
    M llvm/include/llvm/Config/llvm-config.h.cmake
    M llvm/include/llvm/Support/Compiler.h

  Log Message:
  -----------
  change to use llvm-config.h.cmake


Compare: https://github.com/llvm/llvm-project/compare/aa2a5abff7b3...cf02983f4375

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications