replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Handle -mno-<feature> in target attribute strings by replacing the 

-mno- with a -<feature> to match how we handle this in the rest
of the frontend.

llvm-svn: 239581
This commit is contained in:
Eric Christopher 2015-06-12 01:35:58 +00:00
parent 64a247b68b
commit 4dfe075f93
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/CodeGen/CGCall.cpp
View File

@ -1515,8 +1515,10 @@ void CodeGenModule::ConstructAttributeList(const CGFunctionInfo &FI,
else if (Feature.startswith("tune="))
// We don't support cpu tuning this way currently.
;
else
Features.push_back("+" + Feature.str());
else if (Feature.startswith("mno-"))
Features.push_back("-" + Feature.split("-").second.str());
else
Features.push_back("+" + Feature.str());
}
}
}

4
clang/test/CodeGen/attr-target.c
View File

@ -6,6 +6,8 @@ int __attribute__((target("avx,sse4.2,arch=ivybridge"))) foo(int a) { return 4;
int __attribute__((target("tune=sandybridge"))) walrus(int a) { return 4; }
int __attribute__((target("mno-sse2"))) echidna(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.
@ -13,6 +15,8 @@ int bar(int a) { return baz(a) + foo(a); }
// CHECK: foo{{.*}} #1
// We ignore the tune attribute so walrus should be identical to baz and bar.
// CHECK: walrus{{.*}} #0
// CHECK: echidna{{.*}} #2
// 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"
// CHECK: #2 = {{.*}}"target-cpu"="x86-64" "target-features"="+sse,+sse2,-sse2"