[PATCH] D145231: [SCEV] Support mul in SCEVLoopGuardRewriter
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 3 06:16:52 PST 2023
nikic added a comment.
We could also rewrite all types like this: https://github.com/llvm/llvm-project/commit/137e813953991caa93cfe439808618045a3e6311
There is a bit of compile time impact from doing that (though in part it's code size correlated): http://llvm-compile-time-tracker.com/compare.php?from=e00c73c856a325008afead10cfb3e9d0fc4a1e41&to=137e813953991caa93cfe439808618045a3e6311&stat=instructions:u
And code size seems to consistently increase: http://llvm-compile-time-tracker.com/compare.php?from=e00c73c856a325008afead10cfb3e9d0fc4a1e41&to=137e813953991caa93cfe439808618045a3e6311&stat=size-text Not sure if that means something is going wrong, or all optimizations this enables happen to be size increasing.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145231/new/
https://reviews.llvm.org/D145231
More information about the llvm-commits
mailing list