[PATCH] D152876: [InstCombine] Expand `foldBinOpShiftWithShift` to handle multiple binops

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 13 23:56:13 PDT 2023


nikic added a comment.

Is it possible to handle just the case `binop(optional_binop1(shift), optional_binop2(shift))`? I believe that's all that is necessary for the motivating cases and should allow a much cleaner implementation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152876



More information about the llvm-commits mailing list