[PATCH] D144382: [SCEV] Increase FoldID bits size cover common cases.

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 18:39:21 PST 2023


vitalybuka accepted this revision.
vitalybuka added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ScalarEvolution.h:1300
   class FoldID {
-    SmallVector<unsigned, 4> Bits;
+    SmallVector<unsigned, 6> Bits;
 
----------------
it's precisely 5 for 64 and 3 for 32

https://github.com/llvm/llvm-project/commit/f1124364336b8bfa341e8a39caba1170205cb6c3

its evaluates to the same improvement.
Either way prevent SmallVector from using heap,


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144382/new/

https://reviews.llvm.org/D144382



More information about the llvm-commits mailing list