This is an archive of the discontinued LLVM Phabricator instance.

[clang][CodeGen] Break up TargetInfo.cpp [3/8]
ClosedPublic

Authored by barannikov88 on Apr 12 2023, 2:27 AM.

Details

Summary

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.

Diff Detail

Event Timeline

barannikov88 created this revision.Apr 12 2023, 2:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2023, 2:27 AM
barannikov88 published this revision for review.Apr 12 2023, 3:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2023, 3:35 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

clang-format

efriedma accepted this revision.Apr 27 2023, 11:18 AM
efriedma added a subscriber: efriedma.

LGTM

This revision is now accepted and ready to land.Apr 27 2023, 11:18 AM
barannikov88 retitled this revision from [clang][CodeGen] Break up TargetInfo.cpp [3/6] to [clang][CodeGen] Break up TargetInfo.cpp [3/7].May 9 2023, 12:19 AM
barannikov88 retitled this revision from [clang][CodeGen] Break up TargetInfo.cpp [3/7] to [clang][CodeGen] Break up TargetInfo.cpp [3/8].May 9 2023, 11:31 AM
This revision was landed with ongoing or failed builds.May 19 2023, 1:55 PM
This revision was automatically updated to reflect the committed changes.