[PATCH] D111774: [SCEV][NFC] Reduce memory footprint & compile time via DFS refactoring
Max Kazantsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 13 21:42:57 PDT 2021
mkazantsev created this revision.
mkazantsev added reviewers: reames, nikic, lebedev.ri, fhahn, efriedma.
Herald added subscribers: javed.absar, hiraditya.
mkazantsev requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Current implementations of DFS in SCEV check unique-visited of traversed
values on pop, and not on push. As result, the same value may be pushed
multiple times. This theoretically should increase the size of worklist (and
therefore memory footprint during DFS) and also compile time on these
meaningless operations.
This patch reworks the DFS strategy to check uniqueness before push.
Should be NFC.
https://reviews.llvm.org/D111774
Files:
llvm/lib/Analysis/ScalarEvolution.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D111774.379606.patch
Type: text/x-patch
Size: 3881 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211014/059584de/attachment.bin>
More information about the llvm-commits
mailing list