[PATCH] D138489: [tsan] Add tsan support for loongarch64

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 22 17:05:32 PST 2022


MaskRay added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp:269
   val = FIRST_32_SECOND_64(1152, 1776);
+#elif SANITIZER_LOONGARCH64
+  val = 1936;
----------------
We should add new code in this fallback function. The generic code handles all new ports.

It's incorrect to assume that sizeof(pthread) stays 1936 as new glibc versions can add more fields.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138489/new/

https://reviews.llvm.org/D138489



More information about the cfe-commits mailing list