[PATCH] D63321: [SVE][IR] Scalable Vector IR Type with pr42210 fix

[Nico Weber via Phabricator via llvm-commits]

thakis added inline comments.


================
Comment at: lib/IR/Verifier.cpp:343
+          if (VTy->isScalable())
+            CheckFailed("Structs cannot contain scalable vectors", STy, &M);
+    }
----------------
thakis wrote:
> That's still a lot of type walking. Did you have a chance to measure time to thinlto-link some large-ish target with and without this change?
Just saw your comment on https://bugs.llvm.org/show_bug.cgi?id=42210 . That seems fine to me. (I usually do min-of-3 when timing things, to reduce effects of caches.) So from my point of view this is ready for relanding.


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

https://reviews.llvm.org/D63321