[all-commits] [llvm/llvm-project] 95bde4: [MSan] Pass -fsanitize-ignorelist to the instrumen...

Alexander Richardson via All-commits all-commits at lists.llvm.org
Wed Mar 6 22:14:37 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 95bde4b9d6fe258ff7fea6da5024b2a8813c1e0c
      https://github.com/llvm/llvm-project/commit/95bde4b9d6fe258ff7fea6da5024b2a8813c1e0c
  Author: Alexander Richardson <alexrichardson at google.com>
  Date:   2024-03-06 (Wed, 06 Mar 2024)

  Changed paths:
    M compiler-rt/lib/msan/tests/CMakeLists.txt

  Log Message:
  -----------
  [MSan] Pass -fsanitize-ignorelist to the instrumented libcxxabi

This ensures that the MSan unit tests are able to pass with an
uninstrumented libunwind. We need to avoid instrumentation for
__gxx_personality_v0, which is part of the default msan_ignorelist.txt
that is installed into the resource directory. However, if we are trying
to test the just-built libraries, this global ignore list may not be
present yet, so we still instrument the function.

Arguably this function should not be on the default ignore list since it
is only a problem when building libcxxabi with MSan instrumentation and
without an instrumented libunwind, so maybe the logic should really be
part of the libcxxabi build. However, that could be done as a follow-up.

See 2f856a36e0b270b184051d10a18d4b4238b4c033 for more context.

Reviewed By: vitalybuka

Pull Request: https://github.com/llvm/llvm-project/pull/83652



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


More information about the All-commits mailing list