barannikov88 added inline comments.

================
Comment at: clang/lib/CodeGen/CGAtomic.cpp:90
+        StoragePtr = CGF.Builder.CreateAddrSpaceCast(
+            StoragePtr, llvm::PointerType::getUnqual(CGF.getLLVMContext()),
             "atomic_bitfield_base");
----------------
I suppose address space shouldn't be dropped here, but this is what the 
original code does.



================
Comment at: clang/lib/CodeGen/ItaniumCXXABI.cpp:1473
                                                   QualType SrcRecordTy,
                                                   QualType DestTy) {
   auto *ClassDecl =
----------------
`DestTy` has become unused in both implementations. I'm not sure if I should 
remove it. It is always a [cv-qualified] `void *`.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153694/new/

https://reviews.llvm.org/D153694

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to