[PATCH] D113179: [Passes] Move AggressiveInstCombine after InstCombine

Mon Nov 22 12:06:46 PST 2021

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

In D113179#3146480 <https://reviews.llvm.org/D113179#3146480>, @RKSimon wrote:

> any more comments? @lebedev.ri @spatel

I'll make my earlier acceptance official here in Phab. 
I can't find any earlier reviews/comments to justify the ordering, there are no regression tests to support it, and we have evidence that this (new/proposed) ordering helps at least one case (and we still have a bug tracking the root cause).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113179