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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to