[PATCH] D145223: [InstCombine] Combine binary operator of two phi node

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 01:44:09 PST 2023


StephenFan added a comment.

In D145223#4166573 <https://reviews.llvm.org/D145223#4166573>, @xbolva00 wrote:

> This optimization can be more general, why only add?

Yes, it can be more general. But I want to solve the issue first and left a TODO. Or if you strongly recommend implementing other binary operators in this patch, I am willing to do it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145223



More information about the llvm-commits mailing list