[llvm] ef4cc70 - [X86] Remove assert for missing features from X86::getImpliedFeatures

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 7 00:18:51 PDT 2020


Author: Craig Topper
Date: 2020-07-07T00:18:01-07:00
New Revision: ef4cc70f3ed2a91e0a48c6448c517c3ba34c2846

URL: https://github.com/llvm/llvm-project/commit/ef4cc70f3ed2a91e0a48c6448c517c3ba34c2846
DIFF: https://github.com/llvm/llvm-project/commit/ef4cc70f3ed2a91e0a48c6448c517c3ba34c2846.diff

LOG: [X86] Remove assert for missing features from X86::getImpliedFeatures

This is failing on the bots. Remove while I try to figure out
what feature I missed in the table.

Added: 
    

Modified: 
    llvm/lib/Support/X86TargetParser.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Support/X86TargetParser.cpp b/llvm/lib/Support/X86TargetParser.cpp
index 5e4f62d8a1d6..12182179fe45 100644
--- a/llvm/lib/Support/X86TargetParser.cpp
+++ b/llvm/lib/Support/X86TargetParser.cpp
@@ -558,8 +558,6 @@ void llvm::X86::getImpliedFeatures(
   auto I = llvm::find_if(
       FeatureInfos, [&](const FeatureInfo &FI) { return FI.Name == Feature; });
   if (I == std::end(FeatureInfos)) {
-    // This shouldn't happen, but handle it gracefully for release builds.
-    assert(false && "Feature not in table!");
     return;
   }
 


        


More information about the llvm-commits mailing list