[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