[PATCH] D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's BTC value

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 13:25:01 PDT 2020


efriedma added a comment.

Added a related comment to D85737 <https://reviews.llvm.org/D85737>; probably makes sense to continue that discussion there.



================
Comment at: llvm/lib/Target/ARM/MVETailPredication.cpp:624
+    for (auto &I : *BB) {
+      if (I.getOpcode() != Instruction::Add || &I != BTC)
+        continue;
----------------
Why are we searching the basic block here, instead of just using `dyn_cast<Instruction>(BTC)`?


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

https://reviews.llvm.org/D86074



More information about the llvm-commits mailing list