[all-commits] [llvm/llvm-project] ce0c41: [rtsan][NFC] Move assertions and diagnostics into ...

[davidtrevelyan via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ce0c41cb0cfe9a48b6ab48784453503d33420763
      https://github.com/llvm/llvm-project/commit/ce0c41cb0cfe9a48b6ab48784453503d33420763
  Author: davidtrevelyan <davidtrevelyan at users.noreply.github.com>
  Date:   2024-09-20 (Fri, 20 Sep 2024)

  Changed paths:
    M compiler-rt/lib/rtsan/CMakeLists.txt
    M compiler-rt/lib/rtsan/rtsan.cpp
    A compiler-rt/lib/rtsan/rtsan_assertions.cpp
    A compiler-rt/lib/rtsan/rtsan_assertions.h
    M compiler-rt/lib/rtsan/rtsan_context.cpp
    M compiler-rt/lib/rtsan/rtsan_context.h
    A compiler-rt/lib/rtsan/rtsan_diagnostics.cpp
    A compiler-rt/lib/rtsan/rtsan_diagnostics.h
    R compiler-rt/lib/rtsan/rtsan_stack.cpp
    R compiler-rt/lib/rtsan/rtsan_stack.h
    M compiler-rt/lib/rtsan/tests/rtsan_test_context.cpp

  Log Message:
  -----------
  [rtsan][NFC] Move assertions and diagnostics into own impl files (#109500)

In preparation for providing more information to rtsan's diagnostics
output (via `__rtsan_expect_not_realtime`), this PR separates out all
logic for i) making rtsan's assertions about real-time context state and
ii) displaying diagnostics to the user - disentangling them both from
the rtsan `Context`.

We'll follow up this PR with a simplification to the unit tests that
reflect this new separation.



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