[PATCH] D30502: [DAGCombiner] fold binops with constant into select-of-constants

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 1 13:50:07 PST 2017

Previous message: [PATCH] D30502: [DAGCombiner] fold binops with constant into select-of-constants
Next message: [PATCH] D30502: [DAGCombiner] fold binops with constant into select-of-constants
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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

LGTM.

> Or is it ok to assume this is universally good for the currently handled binop types?

Okay for now; if someone eventually trips over it, we can deal with it then.


https://reviews.llvm.org/D30502

Previous message: [PATCH] D30502: [DAGCombiner] fold binops with constant into select-of-constants
Next message: [PATCH] D30502: [DAGCombiner] fold binops with constant into select-of-constants
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list