diff options
| author | Eric Christopher <echristo@gmail.com> | 2015-03-25 22:09:26 +0000 |
|---|---|---|
| committer | Eric Christopher <echristo@gmail.com> | 2015-03-25 22:09:26 +0000 |
| commit | f43966f38df79296cddf216e1b467483045aa8d9 (patch) | |
| tree | a1391000de6e03d5f1ae98d10bdd782f85cefb59 /clang/lib/Basic/Targets.cpp | |
| parent | 533849c615cd39257cb53550d3a2162405b6d63c (diff) | |
| download | bcm5719-llvm-f43966f38df79296cddf216e1b467483045aa8d9.tar.gz bcm5719-llvm-f43966f38df79296cddf216e1b467483045aa8d9.zip | |
Make the msse4/mno-sse4 flags aliases of the maximum sse values
they enable/disable.
This fixes two things:
a) sse4 isn't actually a target feature, don't treat it as one.
b) we weren't correctly disabling sse4.1 when we'd pass -mno-sse4
after enabling it, thus passing preprocessor directives and
(soon) passing the function attribute as well when we shouldn't.
llvm-svn: 233223
Diffstat (limited to 'clang/lib/Basic/Targets.cpp')
| -rw-r--r-- | clang/lib/Basic/Targets.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/clang/lib/Basic/Targets.cpp b/clang/lib/Basic/Targets.cpp index b1a245e1b20..c8fa0cce25f 100644 --- a/clang/lib/Basic/Targets.cpp +++ b/clang/lib/Basic/Targets.cpp @@ -2580,11 +2580,6 @@ void X86TargetInfo::setXOPLevel(llvm::StringMap<bool> &Features, XOPEnum Level, void X86TargetInfo::setFeatureEnabledImpl(llvm::StringMap<bool> &Features, StringRef Name, bool Enabled) { - // FIXME: This *really* should not be here. We need some way of translating - // options into llvm subtarget features. - if (Name == "sse4") - Name = "sse4.2"; - Features[Name] = Enabled; if (Name == "mmx") { |
