Author: Haojian Wu
Date: 2022-07-01T21:25:46+02:00
New Revision: bbcd8e5271f4af3adc28e458642a0dc65d253acd

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

LOG: [pseudo] NFC, polish the fix of c99827349927a44334f2b04139168efd0bc87cd3

Added: 
    

Modified: 
    clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h 
b/clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h
index 664a1a3206fe6..d480956704960 100644
--- a/clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h
+++ b/clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h
@@ -140,11 +140,9 @@ class LRTable {
   //     // ...apply reduce...
   //   }
   llvm::ArrayRef<RuleID> getReduceRules(StateID State) const {
-    if (ReduceOffset[State] >= Reduces.size())
-      return {};
-    size_t Length = ReduceOffset[State + 1] - ReduceOffset[State];
-    return llvm::makeArrayRef(&Reduces[ReduceOffset[State]],
-                              Length);
+    assert(State + 1u < ReduceOffset.size());
+    return llvm::makeArrayRef(Reduces.data() + ReduceOffset[State],
+                              Reduces.data() + ReduceOffset[State+1]);
   }
   // Returns whether Terminal can follow Nonterminal in a valid source file.
   bool canFollow(SymbolID Nonterminal, SymbolID Terminal) const {


        
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to