barannikov88 created this revision. Herald added a project: All. barannikov88 added reviewers: rjmccall, aaron.ballman, erichkeane. barannikov88 published this revision for review. Herald added a project: clang. Herald added a subscriber: cfe-commits.
Don't derive from `DefaultTargetCodeGenInfo`. This class is going to stay in `TargetInfo.cpp`, whereas its derivants are going to be moved to separate translation units. Just derive from the base `TargetCodeGenInfo` class instead. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D148091 Files: clang/lib/CodeGen/TargetInfo.cpp Index: clang/lib/CodeGen/TargetInfo.cpp =================================================================== --- clang/lib/CodeGen/TargetInfo.cpp +++ clang/lib/CodeGen/TargetInfo.cpp @@ -5064,9 +5064,10 @@ llvm::Value *Address) const override; }; -class PPC64TargetCodeGenInfo : public DefaultTargetCodeGenInfo { +class PPC64TargetCodeGenInfo : public TargetCodeGenInfo { public: - PPC64TargetCodeGenInfo(CodeGenTypes &CGT) : DefaultTargetCodeGenInfo(CGT) {} + PPC64TargetCodeGenInfo(CodeGenTypes &CGT) + : TargetCodeGenInfo(std::make_unique<DefaultABIInfo>(CGT)) {} int getDwarfEHStackPointer(CodeGen::CodeGenModule &M) const override { // This is recovered from gcc output. @@ -8599,10 +8600,10 @@ namespace { -class TCETargetCodeGenInfo : public DefaultTargetCodeGenInfo { +class TCETargetCodeGenInfo : public TargetCodeGenInfo { public: TCETargetCodeGenInfo(CodeGenTypes &CGT) - : DefaultTargetCodeGenInfo(CGT) {} + : TargetCodeGenInfo(std::make_unique<DefaultABIInfo>(CGT)) {} void setTargetAttributes(const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const override;
Index: clang/lib/CodeGen/TargetInfo.cpp =================================================================== --- clang/lib/CodeGen/TargetInfo.cpp +++ clang/lib/CodeGen/TargetInfo.cpp @@ -5064,9 +5064,10 @@ llvm::Value *Address) const override; }; -class PPC64TargetCodeGenInfo : public DefaultTargetCodeGenInfo { +class PPC64TargetCodeGenInfo : public TargetCodeGenInfo { public: - PPC64TargetCodeGenInfo(CodeGenTypes &CGT) : DefaultTargetCodeGenInfo(CGT) {} + PPC64TargetCodeGenInfo(CodeGenTypes &CGT) + : TargetCodeGenInfo(std::make_unique<DefaultABIInfo>(CGT)) {} int getDwarfEHStackPointer(CodeGen::CodeGenModule &M) const override { // This is recovered from gcc output. @@ -8599,10 +8600,10 @@ namespace { -class TCETargetCodeGenInfo : public DefaultTargetCodeGenInfo { +class TCETargetCodeGenInfo : public TargetCodeGenInfo { public: TCETargetCodeGenInfo(CodeGenTypes &CGT) - : DefaultTargetCodeGenInfo(CGT) {} + : TargetCodeGenInfo(std::make_unique<DefaultABIInfo>(CGT)) {} void setTargetAttributes(const Decl *D, llvm::GlobalValue *GV, CodeGen::CodeGenModule &M) const override;
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits