[PATCH] D93427: [test] Cleanup some CGSCCPassManager tests

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 16 16:02:17 PST 2020


aeubanks created this revision.
aeubanks added reviewers: asbirlea, rnk.
aeubanks requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Don't iterate over SCC as we potentially modify it.
Verify module (and fix some broken ones).
Only run pass once and make sure that it's actually run.
Rename tests to just end in a number since I'm planning on adding a
bunch more which won't have good individual names. Instead, add comments
on the transformations that each test does.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93427

Files:
  llvm/unittests/Analysis/CGSCCPassManagerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93427.312324.patch
Type: text/x-patch
Size: 7457 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201217/66a52e64/attachment.bin>


More information about the llvm-commits mailing list