[PATCH] D60224: [TargetLowering] Extend bool args to inline-asm according to getBooleanType

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 10:46:46 PDT 2019


nickdesaulniers added a comment.

Otherwise, if we didn't want to change the semantics of `ConstantInt::getSExtValue()`, it looks like there's ~145 call sites that would have to be updated/changed.  Thoughts?


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