[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