[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