[PATCH] D129769: [LSR] Create SCEVExpander earlier, use member isSafeToExpand() (NFC)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 06:29:35 PDT 2022


nikic created this revision.
nikic added reviewers: fhahn, reames.
Herald added subscribers: javed.absar, hiraditya.
Herald added a project: All.
nikic requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This is a followup to D129630 <https://reviews.llvm.org/D129630>, which switches LSR to the member isSafeToExpand() variant, and removes the freestanding function.

This is done by creating the SCEVExpander early (already during the analysis phase). Because the SCEVExpander is now available for the whole lifetime of LSRInstance, I've also made it into a member variable, rather than passing it around in even more places.


https://reviews.llvm.org/D129769

Files:
  llvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
  llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp
  llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129769.444629.patch
Type: text/x-patch
Size: 11570 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220714/05089ae4/attachment.bin>


More information about the llvm-commits mailing list