From: David Majnemer <david.majnemer@gmail.com>
Date: Sun, 11 Jan 2015 09:57:13 +0000 (+0000)
Subject: Basic: [asmSymbolicName] follows the same rule as numbers in asm inputs
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e6253b8c0978f0161420f84d783e427bff25cd03;p=clang

Basic: [asmSymbolicName] follows the same rule as numbers in asm inputs

Input constraints like "0" and "[foo]" should be treated the same when
it comes to their corresponding output constraint.

This fixes PR21850.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@225605 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Basic/TargetInfo.cpp b/lib/Basic/TargetInfo.cpp
index 15b8c83d1b..535c87ec11 100644
--- a/lib/Basic/TargetInfo.cpp
+++ b/lib/Basic/TargetInfo.cpp
@@ -583,6 +583,10 @@ bool TargetInfo::validateInputConstraint(ConstraintInfo *OutputConstraints,
       if (Info.hasTiedOperand() && Info.getTiedOperand() != Index)
         return false;
 
+      // A number must refer to an output only operand.
+      if (OutputConstraints[Index].isReadWrite())
+        return false;
+
       Info.setTiedOperand(Index, OutputConstraints[Index]);
       break;
     }
diff --git a/test/Sema/asm.c b/test/Sema/asm.c
index 10d845e3d0..f386f520be 100644
--- a/test/Sema/asm.c
+++ b/test/Sema/asm.c
@@ -190,3 +190,10 @@ void fn4() {
           : "=r"(l)
           : "#m"(l)); // expected-error {{invalid input constraint '#m' in asm}}
 }
+
+void fn5() {
+  int l;
+    __asm__(""
+          : [g] "+r"(l)
+          : "[g]"(l)); // expected-error {{invalid input constraint '[g]' in asm}}
+}