[PATCH] D40898: [Sanitizers] Basic sanitizer Solaris support (PR 33274)

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 12 11:45:11 PST 2017


alekseyshl accepted this revision.
alekseyshl added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/sanitizer_common/sanitizer_mutex.h:99
+  // Solaris mutex_t has a member that requires 64-bit alignment.
+  ALIGNED(8)
+#endif
----------------
krytarowski wrote:
> I would remove ifdef here.
Yep, let's remove the #ifdef, but keep the comment.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D40898





More information about the llvm-commits mailing list