[PATCH] D60943: Delay diagnosing "n" constraint until after inlining

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 26 15:40:47 PDT 2019


rsmith added a comment.

In D60943#1475038 <https://reviews.llvm.org/D60943#1475038>, @void wrote:

> In D60943#1474926 <https://reviews.llvm.org/D60943#1474926>, @rsmith wrote:
>
> > Is `"n"` really special in this regard, or is it just the first one that we've encountered?
>
>
> I think it's just the first one we've encountered. There could be other constraints which require immediates, but they're probably target-specific. E.g. `e` and `K` for x86.


OK, then I think we should apply whatever we do here to all constraints for which `Info.requiresImmediateConstant()` is `true`.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60943/new/

https://reviews.llvm.org/D60943





More information about the cfe-commits mailing list