[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`.
CHANGES SINCE LAST ACTION
More information about the cfe-commits