sdesmalen marked an inline comment as done.
sdesmalen added a comment.

Thanks for reviewing!



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


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