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

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ScalarEvolution.h:1300
   class FoldID {
-    SmallVector<unsigned, 4> Bits;
+    SmallVector<unsigned, 6> Bits;
 
----------------
vitalybuka wrote:
> 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,
https://llvm-compile-time-tracker.com/compare.php?from=a53d940cee6f281ef1a20d4f0fb39b23b4e98614&to=f1124364336b8bfa341e8a39caba1170205cb6c3&stat=instructions:u


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144382