[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