[PATCH] D43146: [builtin] Update c?zdi2 to cover Linux/sparc

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 07:52:29 PST 2018


JDevlieghere added a comment.

In https://reviews.llvm.org/D43146#1003907, @jrtc27 wrote:

> Actually this might be ok on sparc32? I assume `__builtin_ctz` will instead resolve to `_ctzsi2` being a 32-bit architecture? In which case, shouldn't the `__riscv__` one be checking  `__SIZEOF_POINTER__ == 8`?


Feel free to commandeer this revision with what you think should be the proper solution. As Hans mentioned in https://llvm.org/PR36318 we should do it soon if we want to have this in the 6.0 release.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D43146





More information about the llvm-commits mailing list