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

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 14 09:34:23 PDT 2019


thakis added inline comments.


================
Comment at: lib/IR/Verifier.cpp:343
+          if (VTy->isScalable())
+            CheckFailed("Structs cannot contain scalable vectors", STy, &M);
+    }
----------------
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?


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

https://reviews.llvm.org/D63321





More information about the llvm-commits mailing list