[PATCH] D110908: [NPM] Automatic 'opt' pipeline reducer script.

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks accepted this revision.
aeubanks added a comment.
This revision is now accepted and ready to land.

looks good, thanks!



================
Comment at: llvm/utils/reduce_pipeline.py:45
+parser.add_argument(
+    '--expand-passes',
+    action='store_true',
----------------
I think automatically doing this would be a bit simpler, but not a huge deal


================
Comment at: llvm/utils/reduce_pipeline.py:112
+    [lstA, lstB] = pipeline.split(lst, idx)
+    lstA = pipeline.prune(lstA)
+    lstB = pipeline.prune(lstB)
----------------
very unlikely that this will matter, but perhaps pruning empty pass managers might cause crashes to go away, checking before pruning would be nice


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110908/new/

https://reviews.llvm.org/D110908