[PATCH] D55616: Emit ASM input in a constant context

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 11 17:58:12 PST 2019


void added a comment.

> Apologies - the value seems to indeed overflow, but I'm still very confused how this was affected by this change.

What's different is that `setRequiresImmediate` is being set on the constraint where before it wasn't. If no range values are supplied (like in this patch), it defaults to `INT_MIN` and `INT_MAX`. From GNU's documentation, `n` accepts an integer, which we're treating as signed.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55616





More information about the cfe-commits mailing list