[PATCH] D30708: [ARM] SMULW [T|B] DAG Combine

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 17:49:06 PDT 2017


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM.

> This makes the interface very similar to TLO::SimplifyDemandedBits, do you know why that function is set to assume a single use?

The function has exactly one user in lib/Target/AMDGPU/AMDGPUISelLowering.cpp, and the behavior is necessary there.  (I haven't really carefully considered the consequences of it.)


https://reviews.llvm.org/D30708





More information about the llvm-commits mailing list