From: Eric Christopher <echristo@gmail.com>
Date: Fri, 12 Jun 2015 01:35:56 +0000 (+0000)
Subject: Add support for tune= to the target attribute support by ignoring it.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=25d6352b9e987f39d238f97ca8cf3aac684835ea;p=clang

Add support for tune= to the target attribute support by ignoring it.

We don't currently support the -mtune option in any useful way
so ignoring the annotation is fine.

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

diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp
index 894391c2ab..424997d125 100644
--- a/lib/CodeGen/CGCall.cpp
+++ b/lib/CodeGen/CGCall.cpp
@@ -1512,6 +1512,9 @@ void CodeGenModule::ConstructAttributeList(const CGFunctionInfo &FI,
           // While we're here iterating check for a different target cpu.
           if (Feature.startswith("arch="))
             TargetCPU = Feature.split("=").second;
+	  else if (Feature.startswith("tune="))
+	    // We don't support cpu tuning this way currently.
+	    ;
 	  else
 	    Features.push_back("+" + Feature.str());
 	}
diff --git a/test/CodeGen/attr-target.c b/test/CodeGen/attr-target.c
index dbf00d72e4..2e548cc701 100644
--- a/test/CodeGen/attr-target.c
+++ b/test/CodeGen/attr-target.c
@@ -4,11 +4,15 @@ int baz(int a) { return 4; }
 
 int __attribute__((target("avx,sse4.2,arch=ivybridge"))) foo(int a) { return 4; }
 
+int __attribute__((target("tune=sandybridge"))) walrus(int a) { return 4; }
+
 int bar(int a) { return baz(a) + foo(a); }
 
 // Check that we emit the additional subtarget and cpu features for foo and not for baz or bar.
 // CHECK: baz{{.*}} #0
 // CHECK: foo{{.*}} #1
+// We ignore the tune attribute so walrus should be identical to baz and bar.
+// CHECK: walrus{{.*}} #0
 // CHECK: bar{{.*}} #0
 // CHECK: #0 = {{.*}}"target-cpu"="x86-64" "target-features"="+sse,+sse2"
 // CHECK: #1 = {{.*}}"target-cpu"="ivybridge" "target-features"="+sse,+sse2,+avx,+sse4.2"