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