[PATCH] D107058: [SLP]Fix a crash in gathered loads analysis.

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 29 05:41:52 PDT 2021


sdesmalen added a comment.

Thanks for fixing!



================
Comment at: llvm/test/Transforms/SLPVectorizer/AArch64/loads-gather-crash.ll:4
+
+define void @foo() local_unnamed_addr {
+; CHECK-LABEL: @foo(
----------------
Could you name the test to something more descriptive, e.g. `gather-load-min-required-vf-2.ll` (feel free to come up with something better) and also add a comment explaining why the code should not be slp-vectorized?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107058



More information about the llvm-commits mailing list