efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM with one minor comment



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7739
+
+    // Index needs to be passed as scaled offset.
+    llvm::Type *MemEltTy = SVEBuiltinMemEltTy(TypeFlags);
----------------
`} else {`, so you aren't multiplying by zero?


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

https://reviews.llvm.org/D78677



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to