[PATCH] D73411: [InstCombine] Process newly inserted instructions in the correct order

Wed Jan 29 12:35:27 PST 2020

nikic marked an inline comment as done.
nikic added inline comments.

================
Comment at: llvm/test/Transforms/InstCombine/getelementptr.ll:223
+; CHECK-NEXT:    [[C:%.*]] = icmp eq <2 x i64> [[A_OFFS]], zeroinitializer
 ; CHECK-NEXT:    ret <2 x i1> [[C]]
 ;
----------------
spatel wrote:
> nikic wrote:
> > This issue is tracked at https://bugs.llvm.org/show_bug.cgi?id=44588. I got the impression that @spatel is planning to work on this at some point, but if desired I can also look into fixing this regression first.
> I took a look at that, and I think we need an analysis enhancement first:
> D73549
D73575 and D73647 have addressed this regression. We now get a marginally better result here.

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

https://reviews.llvm.org/D73411