[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