[CIR] [CodeGen] Handle NYI in CIRGenModule::tryEmitBaseDestructorAsAlias

Author: ChuanqiXu9

clang/lib/CIR/CodeGen/CIRGenCXX.cpp

@@ -145,7 +145,8 @@ bool CIRGenModule::tryEmitBaseDestructorAsAlias(const CXXDestructorDecl *D) {
     // members with attribute "AlwaysInline" and expect no reference to
     // be generated. It is desirable to reenable this optimisation after
     // corresponding LLVM changes.
-    llvm_unreachable("NYI");
+    addReplacement(MangledName, Aliasee);
+    return false;
   }
 
   // If we have a weak, non-discardable alias (weak, weak_odr), like an
@@ -154,7 +155,8 @@ bool CIRGenModule::tryEmitBaseDestructorAsAlias(const CXXDestructorDecl *D) {
   // symbol reference from another TU. The other TU must also mark the
   // referenced symbol as weak, which we cannot rely on.
   if (cir::isWeakForLinker(Linkage) && getTriple().isOSBinFormatCOFF()) {
-    llvm_unreachable("NYI");
+    assert(false && "please sent a PR with a test and remove this.\n");
+    return true;
   }
 
   // If we don't have a definition for the destructor yet or the definition
@@ -168,8 +170,10 @@ bool CIRGenModule::tryEmitBaseDestructorAsAlias(const CXXDestructorDecl *D) {
   // different COMDATs in different TUs. Another option would be to
   // output the alias both for weak_odr and linkonce_odr, but that
   // requires explicit comdat support in the IL.
-  if (cir::isWeakForLinker(TargetLinkage))
-    llvm_unreachable("NYI");
+  if (cir::isWeakForLinker(TargetLinkage)) {
+    assert(false && "please sent a PR with a test and remove this.\n");
+    return true;
+  }
 
   // Create the alias with no name.
   emitAliasForGlobal(MangledName, Entry, AliasDecl, Aliasee, Linkage);

clang/lib/CIR/CodeGen/CIRGenModule.cpp

@@ -3205,6 +3205,40 @@ void CIRGenModule::applyReplacements() {
     auto NewF = dyn_cast<cir::FuncOp>(Replacement);
     assert(NewF && "not implemented");
 
+    // LLVM has opaque pointer but CIR not. So we may have to handle these
+    // different pointer types when performing replacement.
+    auto optionalUseRange = OldF.getSymbolUses(theModule);
+    if (optionalUseRange) {
+      for (auto U : *optionalUseRange) {
+        auto Call = mlir::dyn_cast<cir::CallOp>(U.getUser());
+        if (!Call)
+          continue;
+
+        auto ArgOps = Call.getArgOps();
+        auto FuncArgTypes = NewF.getFunctionType().getInputs();
+        for (unsigned I = 0; I < FuncArgTypes.size(); I++) {
+          if (ArgOps[I].getType() == FuncArgTypes[I])
+            continue;
+
+          auto argPointerTy =
+              mlir::dyn_cast<cir::PointerType>(ArgOps[I].getType());
+          auto funcArgPointerTy =
+              mlir::dyn_cast<cir::PointerType>(FuncArgTypes[I]);
+
+          // If we can't solve it, leave it for the verifier to bail out.
+          if (!argPointerTy || !funcArgPointerTy)
+            continue;
+
+          mlir::OpBuilder::InsertionGuard guard(builder);
+          builder.setInsertionPoint(Call);
+          auto castedArg =
+              builder.createBitcast(Call.getLoc(), ArgOps[I], funcArgPointerTy);
+          // FIXME: Raw API. Wrap this into cir::CallOp.
+          Call.setOperand(I, castedArg);
+        }
+      }
+    }
+
     // Replace old with new, but keep the old order.
     if (OldF.replaceAllSymbolUses(NewF.getSymNameAttr(), theModule).failed())
       llvm_unreachable("internal error, cannot RAUW symbol");

clang/test/CIR/CodeGen/dtor-alias.cpp

@@ -0,0 +1,18 @@
+// FIXME: Remove of -clangir-disable-passes may trigger a memory safe bug in CIR internally during
+// lowering
+// RUN: %clang_cc1 -std=c++17 -triple x86_64-unknown-linux-gnu -O1 \
+// RUN:   -mconstructor-aliases -fclangir -emit-cir %s -o %t.cir \
+// RUN:   -clangir-disable-passes -o %t.cir
+// RUN: FileCheck %s --input-file=%t.cir
+
+namespace {
+struct A {
+  ~A() {}
+};
+
+struct B : public A {};
+}
+
+B x;
+
+// CHECK: cir.call @_ZN12_GLOBAL__N_11AD2Ev({{.*}}) : (!cir.ptr<!ty_28anonymous_namespace293A3AA>) -> ()