[PATCH] D82105: [ARM][MVE] Tail-Prediction: revert multiple get.active.lane.mask intrinsics

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 18 14:47:05 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Target/ARM/MVETailPredication.cpp:169
 
 void MVETailPredication::RevertActiveLaneMask() {
+  if (ActiveLaneMasks.empty())
----------------
I think I'd prefer if you just taught isel legalization to lower llvm.get.active.lane.mask calls, rather than depend on the MVETailPredication pass running before isel.


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

https://reviews.llvm.org/D82105





More information about the llvm-commits mailing list