[PATCH] D60224: [TargetLowering] Extend bool args to inline-asm according to getBooleanType
James Y Knight via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 3 14:03:53 PDT 2019
jyknight added a comment.
After looking at this a bit more, I realized that this is still insufficient. There's *many* more target-specific constraints than just x86 "i" which sign extend constant integer arguments -- and we'll need to apply this change to every one of them, in all targets.
I feel like perhaps there's another better place to do the bool handling, rather than adding it to every constraint's processing?
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60224/new/
https://reviews.llvm.org/D60224
More information about the llvm-commits
mailing list