================ @@ -638,7 +638,7 @@ __uninitialized_allocator_relocate(_Alloc& __alloc, _Tp* __first, _Tp* __last, _ __guard.__complete(); std::__allocator_destroy(__alloc, __first, __last); } else { - __builtin_memcpy(__result, __first, sizeof(_Tp) * (__last - __first)); + __builtin_memcpy((void*)__result, __first, sizeof(_Tp) * (__last - __first)); ---------------- philnik777 wrote:
I think it would be better to make these `static_cast`s to make it obvious that we're casting from a different pointer. Could you also add a comment like `// casting to void* to suppress clang complaining that this is technically UB`? https://github.com/llvm/llvm-project/pull/111434 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits