diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2023-02-17 10:11:40 -0500 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2023-02-17 10:30:15 -0500 |
| commit | 4ecc6af813e2568c3529ea0fe55eaea891d8ab49 (patch) | |
| tree | 7986a0fda5905e45ca70d6dbf04aa2a15532e1af /llvm/unittests/ADT/StringMapTest.cpp | |
| parent | e03d254bbd548d56fba43f805acb5fa31694d602 (diff) | |
| download | llvm-4ecc6af813e2568c3529ea0fe55eaea891d8ab49.zip llvm-4ecc6af813e2568c3529ea0fe55eaea891d8ab49.tar.gz llvm-4ecc6af813e2568c3529ea0fe55eaea891d8ab49.tar.bz2 | |
[InstCombine] create a pass options container and add "use-loop-info" argument
This is a cleanup/modernization patch requested in D144045 to make loop
analysis a proper optional parameter to the pass rather than a
semi-arbitrary value inherited via the pass pipeline.
It's a bit more complicated than the recent patch I started copying from
(D143980) because InstCombine already has an option for MaxIterations
(added with D71145).
I debated just deleting that option, but it was used by a pair of existing
tests, so I put it into a struct (code largely copied from SimplifyCFG's
implementation) to make the code more flexible for future options
enhancements.
I didn't alter the pass manager invocations of InstCombine in this patch
because the patch was already getting big, but that will be a small
follow-up as noted with the TODO comment.
Differential Revision: https://reviews.llvm.org/D144199
Diffstat (limited to 'llvm/unittests/ADT/StringMapTest.cpp')
0 files changed, 0 insertions, 0 deletions