diff options
author | Eric Christopher <echristo@gmail.com> | 2015-08-31 23:19:55 +0000 |
---|---|---|
committer | Eric Christopher <echristo@gmail.com> | 2015-08-31 23:19:55 +0000 |
commit | dec31befeff9fe0bc71fe1ba1ffa502cd04e73ca (patch) | |
tree | e4c3a59298ac258af062de0cf7599dac3fe217e7 /clang/lib/Basic/TargetInfo.cpp | |
parent | 1baec5323bca44f5e79c4d62a36e3c82a2fdc81c (diff) | |
download | bcm5719-llvm-dec31befeff9fe0bc71fe1ba1ffa502cd04e73ca.tar.gz bcm5719-llvm-dec31befeff9fe0bc71fe1ba1ffa502cd04e73ca.zip |
Revert "Pull the target attribute parsing out of CGCall and onto TargetInfo."
This reverts commit r246468 while we figure out what to do about Basic and AST.
llvm-svn: 246508
Diffstat (limited to 'clang/lib/Basic/TargetInfo.cpp')
-rw-r--r-- | clang/lib/Basic/TargetInfo.cpp | 47 |
1 files changed, 0 insertions, 47 deletions
diff --git a/clang/lib/Basic/TargetInfo.cpp b/clang/lib/Basic/TargetInfo.cpp index 2ee705272dd..30378a5a751 100644 --- a/clang/lib/Basic/TargetInfo.cpp +++ b/clang/lib/Basic/TargetInfo.cpp @@ -650,50 +650,3 @@ bool TargetInfo::validateInputConstraint(ConstraintInfo *OutputConstraints, return true; } - -bool TargetInfo::initFeatureMap(llvm::StringMap<bool> &Features, - DiagnosticsEngine &Diags, StringRef CPU, - std::vector<std::string> &FeatureVec) const { - for (const auto &F : FeatureVec) { - const char *Name = F.c_str(); - // Apply the feature via the target. - bool Enabled = Name[0] == '+'; - setFeatureEnabled(Features, Name + 1, Enabled); - } - return true; -} - -TargetInfo::ParsedTargetAttr -TargetInfo::parseTargetAttr(const TargetAttr *TA) const { - std::pair<StringRef, std::vector<std::string>> RetPair; - - // Grab the target attribute string. - StringRef FeaturesStr = TA->getFeatures(); - SmallVector<StringRef, 1> AttrFeatures; - FeaturesStr.split(AttrFeatures, ","); - - // Grab the various features and prepend a "+" to turn on the feature to - // the backend and add them to our existing set of features. - for (auto &Feature : AttrFeatures) { - // Go ahead and trim whitespace rather than either erroring or - // accepting it weirdly. - Feature = Feature.trim(); - - // While we're here iterating check for a different target cpu. - if (Feature.startswith("arch=")) - RetPair.first = Feature.split("=").second.trim(); - else if (Feature.startswith("tune=")) - // We don't support cpu tuning this way currently. - ; - else if (Feature.startswith("fpmath=")) - // TODO: Support the fpmath option this way. It will require checking - // overall feature validity for the function with the rest of the - // attributes on the function. - ; - else if (Feature.startswith("no-")) - RetPair.second.push_back("-" + Feature.split("-").second.str()); - else - RetPair.second.push_back("+" + Feature.str()); - } - return RetPair; -} |