[PATCH] D149448: [Pipelines] Move LTO SCCP and BDCE out of addVectorPasses()

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello. This sounds OK, but apparently it can be pretty beneficial to run BDCE after the second round of unrolling. Unfortunately this may be wrapped up with some downstream compiler changes with a downstream test, it would make it hard to produce a reproducer. It is apparently after the second round of unrolling in this particular test. Otherwise I don't see any issues past that one case.


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

https://reviews.llvm.org/D149448