[PATCH] D125160: [InstCombine] Reassociate add sequences to reduce dependency chains

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 7 05:48:30 PDT 2022


spatel added a comment.

In D125160#3498711 <https://reviews.llvm.org/D125160#3498711>, @xbolva00 wrote:

> https://reviews.llvm.org/D67383 is more general solution, no?

Yes, that's definitely a bigger/better solution for the likely real-world problems. That patch stalled, but I don't think there were any real objections to adding a pass like that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125160



More information about the llvm-commits mailing list