[llvm] a208b68 - [X86TargetParser] Simplify X86_FEATURE_COMPAT assert. NFC

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 21:31:44 PDT 2023


Author: Fangrui Song
Date: 2023-08-24T21:31:39-07:00
New Revision: a208b6840146dde00fcb6ffaaf15718a604e9117

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

LOG: [X86TargetParser] Simplify X86_FEATURE_COMPAT assert. NFC

Added: 
    

Modified: 
    llvm/lib/TargetParser/X86TargetParser.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/TargetParser/X86TargetParser.cpp b/llvm/lib/TargetParser/X86TargetParser.cpp
index 3a36289044fad0..b934f02404ec05 100644
--- a/llvm/lib/TargetParser/X86TargetParser.cpp
+++ b/llvm/lib/TargetParser/X86TargetParser.cpp
@@ -725,13 +725,11 @@ unsigned llvm::X86::getFeaturePriority(ProcessorFeatures Feat) {
 #define X86_FEATURE_COMPAT(ENUM, STR, PRIORITY) PRIORITY,
   unsigned Priorities[] = {
 #include "llvm/TargetParser/X86TargetParser.def"
-      std::numeric_limits<unsigned>::max() // Need to consume last comma.
   };
-  std::array<unsigned, std::size(Priorities) - 1> HelperList;
+  std::array<unsigned, std::size(Priorities)> HelperList;
   std::iota(HelperList.begin(), HelperList.end(), 0);
   assert(std::is_permutation(HelperList.begin(), HelperList.end(),
-                             std::begin(Priorities),
-                             std::prev(std::end(Priorities))) &&
+                             std::begin(Priorities), std::end(Priorities)) &&
          "Priorities don't form consecutive range!");
 #endif
 


        


More information about the llvm-commits mailing list