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

Mikhail Gudim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 15:34:39 PDT 2023


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



More information about the llvm-commits mailing list