================ @@ -3088,9 +3088,6 @@ void ItaniumCXXABI::EmitThreadLocalInitFuncs( CharUnits Align = CGM.getContext().getDeclAlign(VD); Val = Builder.CreateAlignedLoad(Var->getValueType(), Val, Align); } - if (Val->getType() != Wrapper->getReturnType()) - Val = Builder.CreatePointerBitCastOrAddrSpaceCast( ---------------- s-barannikov wrote:
I'm in favor of stripping it. In the end, codebase simplification is the whole point of opaque pointers. This check was added in https://reviews.llvm.org/D5353, which has nothing to do with address spaces. If it ever breaks, it should be easy to fix. https://github.com/llvm/llvm-project/pull/68277 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits