[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