From eddecd3ade5ce1b827dafb84e8137cc6f89576fb Mon Sep 17 00:00:00 2001 From: Florian Hahn Date: Fri, 24 Feb 2023 09:41:52 +0100 Subject: [PATCH] [SCEV] Increase FoldID bits size cover common cases. This should fix the regressions introduced by a53d940cee6f by making sure no dynamic allocations are needed in the common case, while retaining support for arbitrary SCEV expressions. Alternative to D144335. Compile-time impact: * NewPM-O3: -0.11% * NewPM-ReleaseThinLTO: -0.10% *NewPM-ReleaseLTO-g: -0.08% https://llvm-compile-time-tracker.com/compare.php?from=df016a9525e5722dfd1e1e1632cec3ed7b33bc8a&to=c1c64de4a973bcecaddbc4038a539234eb39413b&stat=instructions:u Reviewed By: vitalybuka, nikic Differential Revision: https://reviews.llvm.org/D144382 --- llvm/include/llvm/Analysis/ScalarEvolution.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/include/llvm/Analysis/ScalarEvolution.h b/llvm/include/llvm/Analysis/ScalarEvolution.h index 57107cb..51b67d1 100644 --- a/llvm/include/llvm/Analysis/ScalarEvolution.h +++ b/llvm/include/llvm/Analysis/ScalarEvolution.h @@ -1297,7 +1297,7 @@ public: bool loopIsFiniteByAssumption(const Loop *L); class FoldID { - SmallVector Bits; + SmallVector Bits; public: void addInteger(unsigned long I) { -- 2.7.4