[PATCH] D151758: [Reassociate] Group select of constants together with constants

[Mikhail Gudim via Phabricator via llvm-commits]

mgudim added a comment.

Posting this as a draft to get some feedback / advice.  I am not sure about the following points:

(1) If only one of the arguments of select is constant, putting select next to a constant could be still beneficial. 
(2) How should we handle selects on different conditions?
(3) Would it be better to move selects next to constants in `ReassociateExpression` instead?


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

https://reviews.llvm.org/D151758