[PATCH] D81340: [ARM] Split FPExt loads

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 10:25:38 PDT 2020


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

LGTM with one minor comment.



================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:15016
+                                    Loads[i * 2 + 0], Loads[i * 2 + 1]));
+    Loads = Concats;
+  }
----------------
Instead of looping like this, you can probably just `return DAG.getNode(ISD::CONCAT_VECTORS, DL, ConcatVT, Loads);`.


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

https://reviews.llvm.org/D81340





More information about the llvm-commits mailing list