rjmccall added inline comments.
================ Comment at: clang/lib/Sema/SemaInit.cpp:4297 + ? S.Context.getQualifiedType(T1, T1Quals.withoutAddressSpace()) + : cv1T1; // Not reference-related. Create a temporary and bind to that. ---------------- Should we be rejecting this path immediately if the address space in T1 can't be converted to from the address space of temporaries? ================ Comment at: clang/lib/Sema/SemaInit.cpp:4308 + if (T1Quals.hasAddressSpace()) + Sequence.AddQualificationConversionStep(cv1T1, VK_XValue); + } else ---------------- This should only be `VK_XValue` if we're binding a r-value reference, I think. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95608/new/ https://reviews.llvm.org/D95608 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
