[all-commits] [llvm/llvm-project] a6b806: [TSan][Apple] Fix interceptor build error

Julian Lettner via All-commits all-commits at lists.llvm.org
Fri Jan 24 13:01:11 PST 2025


  Branch: refs/heads/users/yln/tsan-uneclared-identifier-OSSpinLockUnlock
  Home:   https://github.com/llvm/llvm-project
  Commit: a6b80637acde696b1fe23ec7f5caab1540605914
      https://github.com/llvm/llvm-project/commit/a6b80637acde696b1fe23ec7f5caab1540605914
  Author: Julian Lettner <jlettner at apple.com>
  Date:   2025-01-24 (Fri, 24 Jan 2025)

  Changed paths:
    M compiler-rt/lib/tsan/rtl/tsan_interceptors_mac.cpp
    R compiler-rt/lib/tsan/rtl/tsan_spinlock_defs_mac.h

  Log Message:
  -----------
  [TSan][Apple] Fix interceptor build error

In certain cases, the SDK headers declare
`OSSpinLock*` APIs as macros (instead of
functions), so users can be transparently
forwarded to non-deprecated APIs.

When enabled, building of TSan interceptors failed
because these macros interfere with the
interceptor machinery, i.e., they prevent proper
forward declaration of intercepted APIs.

In a previous change [1], we misattributed this to
the deprecation of `OSSpinLock*` APIs.

[1] ae484c21c05668f84b13304c28bc39f753e493de

rdar://143193907


  Commit: f552c55808719c79b76236f931691068b185c827
      https://github.com/llvm/llvm-project/commit/f552c55808719c79b76236f931691068b185c827
  Author: Julian Lettner <jlettner at apple.com>
  Date:   2025-01-24 (Fri, 24 Jan 2025)

  Changed paths:
    M compiler-rt/lib/tsan/rtl/tsan_interceptors_mac.cpp

  Log Message:
  -----------
  [TSan][Apple] Prevent disabling deprecation warnings for entire file

Add missing `#pragma clang diagnostic pop` to
prevent disabling deprecation warnings for the
rest of the file.


Compare: https://github.com/llvm/llvm-project/compare/a6b80637acde%5E...f552c5580871

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