[llvm] GlobalISel: Use G_UADDE when narrowing G_UMULH (PR #97194)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 2 07:38:40 PDT 2024
arsenm wrote:
> > In this case, I am strongly opposed. I don't want the combiner to create more legalization artefacts.
>
> I don't follow. No legalization artifacts are mentioned here. It's an instruction, typically used for legalization, which should have a rich set of optimization combines on it. It is not a legalization artifact, nor are legalization artifacts something to be avoided
Plus this is a separate TODO combine, we're avoiding / delaying doing it by introducing UADDE from the beginning
https://github.com/llvm/llvm-project/pull/97194
More information about the llvm-commits
mailing list