[PATCH] D99029: [RISCV] Don't form MULW for (sext_inreg (mul X, Y), i32)) if the mul has another use.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 31 07:10:42 PDT 2021


asb added a comment.

In D99029#2654346 <https://reviews.llvm.org/D99029#2654346>, @craig.topper wrote:

> I made a dedicated fix for the motivating issue with 5692fc38e0d17abc55a4a84da98f021a1d53d76d <https://reviews.llvm.org/rG5692fc38e0d17abc55a4a84da98f021a1d53d76d> so I'm not as concerned with pushing this patch now.

Ack. The approach of adding a one-use check seems sensible, so if it does make sense to respin and push this further, obviously just ping here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99029



More information about the llvm-commits mailing list