[PATCH] D97292: [ARM] Add FP handling for MVE lane interleaving

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: llvm/lib/Target/ARM/MVELaneInterleavingPass.cpp:127
+  // But expensive extends/truncs are always good to remove. FPExts always
+  // involve extra VCVT's so are always considered to be cheap.
+  for (auto *E : Exts) {
----------------
Can you clarify this a bit? I am no following the "extra VCVT's" => "cheap" reasoning. I am guessing you meant expensive?


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

https://reviews.llvm.org/D97292