[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