[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