[PATCH] D105589: [ARM] Move the creation of VBICIMM and VORRIMM to lowering

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 7 14:11:03 PDT 2021


efriedma added a comment.

If you just want to delay the combine, I'd suggest just adding a `DCI.isAfterLegalizeDAG()` to the existing DAGCombine.  Doing it during lowering can lead to weird results; for example, AArch64 currently does this, and there's testcase where we end up with `orr v0.2s, #0`.


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

https://reviews.llvm.org/D105589



More information about the llvm-commits mailing list