[PATCH] D56057: [X86] Individually simplify both operands of PMULDQ/PMULUDQ using the other entry point of SimplifyDemandedBits that allows the one use check of the root node to be suppressed.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 23 21:06:40 PST 2018
craig.topper added a comment.
The really concerning thing is that this patch does the exact same thing that's done by simplifyI24 in AMDGPUISelLowering.cpp. I assume Simon like myself assumed that established infrastructure like that was doing the right thing and didn't question it too much.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56057/new/
https://reviews.llvm.org/D56057
More information about the llvm-commits
mailing list