From 25d6352b9e987f39d238f97ca8cf3aac684835ea Mon Sep 17 00:00:00 2001
From: Eric Christopher <echristo@gmail.com>
Date: Fri, 12 Jun 2015 01:35:56 +0000
Subject: [PATCH] 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
---
lib/CodeGen/CGCall.cpp | 3 +++
test/CodeGen/attr-target.c | 4 ++++
2 files changed, 7 insertions(+)
diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp
index 894391c2ab5..424997d125e 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 dbf00d72e40..2e548cc701a 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"
--
GitLab