[libc] [lldb] [mlir] [clang-tools-extra] [flang] [clang] [llvm] [compiler-rt] [libcxx] [X86] Add "Ws" constraint and "p" modifier for symbolic address/label reference (PR #77886)

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 14 00:53:14 PST 2024


================
@@ -1418,6 +1418,14 @@ bool X86TargetInfo::validateAsmConstraint(
   case 'O':
     Info.setRequiresImmediate(0, 127);
     return true;
+  case 'W':
+    switch (*++Name) {
+    default:
+      return false;
+    case 's':
+      Info.setAllowsRegister();
----------------
MaskRay wrote:

`setAllowsRegister` is somewhat confusing but its use is correct here (also used by aarch64/riscv `S`).

`setAllowsMemory` seems to not add extra checks.

https://github.com/llvm/llvm-project/pull/77886


More information about the cfe-commits mailing list