[PATCH] D90715: [VPlan] NFC: Change VFRange to take ElementCount

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 5 03:03:31 PST 2020


fhahn accepted this revision.
fhahn added a comment.

LGTM



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:75
+      : Start(Start), End(End) {
+    assert(Start.isScalable() == End.isScalable() &&
+           "Both Start and End should have the same scalable flag");
----------------
Now that we have a constructor here, can we also assert that Start is a power-of-2?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90715



More information about the llvm-commits mailing list