[PATCH] D57401: [DAGCombiner] fold add/sub with bool operand based on target's boolean contents

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: test/CodeGen/PowerPC/select_const.ll:268-269
 }
 
+; FIXME: This could be addi 3, 3, 42
+
----------------
This looks like a missing target-independent pattern; we're ending up with something like "42 - (cond & 1)" instead of "42 + cond".


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57401/new/

https://reviews.llvm.org/D57401