[PATCH] D64713: [InstCombine] X *fast (C ? 1.0 : 0.0) -> C ? X : 0.0
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 15 03:23:17 PDT 2019
foad added a comment.
In D64713#1585110 <https://reviews.llvm.org/D64713#1585110>, @xbolva00 wrote:
> b) yes, one more test never hurts
I meant:
b) The patch doesn't introduce any more uses of hasOneUse() in the source code. Do you think it should?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64713/new/
https://reviews.llvm.org/D64713
More information about the llvm-commits
mailing list