[PATCH] D86377: [tsan] Use large address space mapping on Apple Silicon Macs

Julian Lettner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 24 10:12:34 PDT 2020


yln added a comment.

In D86377#2231641 <https://reviews.llvm.org/D86377#2231641>, @kubamracek wrote:

> Definitely agree that this is very annoying to read. I think it would be best if we were able to change the ifdef to use something like “HAS_48_BIT_ADDRESS_SPACE” and compute that bool separately. @yln, @dvyukov WDYT?

That sounds good to me and even more expressive than what I suggested.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86377



More information about the llvm-commits mailing list