c-rhodes accepted this revision. c-rhodes added a comment. This revision is now accepted and ready to land.
Thanks Craig this is a better solution, LLVM cleans up the store/load coercion nicely but better not to create them in the first place if possible. LGTM cheers ================ Comment at: clang/lib/CodeGen/CGCall.cpp:5749 + // intrinsic to perform the conversion. + if (auto *FixedDst = dyn_cast<llvm::FixedVectorType>(ConvertType(RetTy))) { + llvm::Value *V = CI; ---------------- nit: RetIRTy Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155495/new/ https://reviews.llvm.org/D155495 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits