[PATCH] D75478: [SVE] Make getNumElements assert if the vector is scalable

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 15:39:24 PST 2020


efriedma added a reviewer: sdesmalen.
efriedma added a comment.

https://reviews.llvm.org/B47829 says there are regression test failures; we can't merge until those are fixed.

I think this makes sense.



================
Comment at: llvm/lib/IR/Type.cpp:584
+
+uint64_t SequentialType::getNumElements() const {
+#ifndef NDEBUG
----------------
Please make this inline.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75478





More information about the llvm-commits mailing list