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

Wed Dec 1 10:31:03 PST 2021

anton-afanasyev added a comment.

In D113179#3147047 <https://reviews.llvm.org/D113179#3147047>, @spatel wrote:

> 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).

@lebedev.ri Need your final approval :)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113179