This is an archive of the discontinued LLVM Phabricator instance.

Differential D153704

[openmp] remove initializeRewriteSymbolsLegacyPassPass
ClosedPublic

Authored by sunshaoce on Jun 24 2023, 9:08 AM.

Details

Reviewers
kazu
aeubanks
nikic
jdoerfert
Commits
rGf042890521f3: [openmp] remove initializeRewriteSymbolsLegacyPassPass
Summary

Fix build error caused by D153679

Diff Detail

Event Timeline

sunshaoce created this revision.Jun 24 2023, 9:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 24 2023, 9:08 AM
Herald added subscribers: StephenFan, guansong, yaxunl. · View Herald Transcript
sunshaoce requested review of this revision.Jun 24 2023, 9:08 AM
Herald added a reviewer: jdoerfert. · View Herald TranscriptJun 24 2023, 9:08 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: openmp-commits, jplehr, sstefan1. · View Herald Transcript
Harbormaster completed remote builds in B240973: Diff 534224.Jun 24 2023, 9:11 AM
nikic added inline comments.Jun 24 2023, 9:11 AM
openmp/libomptarget/plugins-nextgen/common/PluginInterface/JIT.cpp
83

You can remove the entire code starting from here.

tianshilei1992 added a subscriber: tianshilei1992.Jun 24 2023, 9:17 AM
tianshilei1992 added inline comments.
openmp/libomptarget/plugins-nextgen/common/PluginInterface/JIT.cpp
83

we don't need to initialize them anymore?

sunshaoce updated this revision to Diff 534225.Jun 24 2023, 9:21 AM
sunshaoce marked an inline comment as done.

Addressed @nikic's comment. Removed Initialize passes.

Herald added a subscriber: aheejin. · View Herald TranscriptJun 24 2023, 9:21 AM
Harbormaster completed remote builds in B240974: Diff 534225.Jun 24 2023, 9:24 AM
nikic accepted this revision.Jun 24 2023, 9:28 AM

LGTM

openmp/libomptarget/plugins-nextgen/common/PluginInterface/JIT.cpp
80

You can remove this check (and the JITTargetInitialized) flag as well.

This revision is now accepted and ready to land.Jun 24 2023, 9:28 AM
sunshaoce updated this revision to Diff 534229.Jun 24 2023, 9:33 AM
sunshaoce marked an inline comment as done.

Removed JITTargetInitialized.

This revision was landed with ongoing or failed builds.Jun 24 2023, 9:35 AM
Closed by commit rGf042890521f3: [openmp] remove initializeRewriteSymbolsLegacyPassPass (authored by sunshaoce). · Explain Why
This revision was automatically updated to reflect the committed changes.
sunshaoce added a commit: rGf042890521f3: [openmp] remove initializeRewriteSymbolsLegacyPassPass.
Harbormaster completed remote builds in B240977: Diff 534229.Jun 24 2023, 9:36 AM

Revision Contents

PathSize
openmp/
libomptarget/
plugins-nextgen/
common/
PluginInterface/
1 line

Diff 534224

openmp/libomptarget/plugins-nextgen/common/PluginInterface/JIT.cpp

Show First 20 Lines • Show All 71 Lines • ▼ Show 20 Lines#ifdef LIBOMPTARGET_JIT_AMDGPU
if (TT.isAMDGPU()) { if (TT.isAMDGPU()) {
LLVMInitializeAMDGPUTargetInfo(); LLVMInitializeAMDGPUTargetInfo();
LLVMInitializeAMDGPUTarget(); LLVMInitializeAMDGPUTarget();
LLVMInitializeAMDGPUTargetMC(); LLVMInitializeAMDGPUTargetMC();
LLVMInitializeAMDGPUAsmPrinter(); LLVMInitializeAMDGPUAsmPrinter();
JITTargetInitialized = true; JITTargetInitialized = true;
} }
#endif #endif
if (!JITTargetInitialized) if (!JITTargetInitialized)
nikicUnsubmitted
Done
ReplyInline Actions

You can remove this check (and the JITTargetInitialized) flag as well.

nikic: You can remove this check (and the JITTargetInitialized) flag as well.
return; return;
// Initialize passes // Initialize passes
nikicUnsubmitted
Done
ReplyInline Actions

You can remove the entire code starting from here.

nikic: You can remove the entire code starting from here.
tianshilei1992Unsubmitted
Not Done
ReplyInline Actions

we don't need to initialize them anymore?

tianshilei1992: we don't need to initialize them anymore?
PassRegistry &Registry = *PassRegistry::getPassRegistry(); PassRegistry &Registry = *PassRegistry::getPassRegistry();
initializeCore(Registry); initializeCore(Registry);
initializeScalarOpts(Registry); initializeScalarOpts(Registry);
initializeVectorization(Registry); initializeVectorization(Registry);
initializeIPO(Registry); initializeIPO(Registry);
initializeAnalysis(Registry); initializeAnalysis(Registry);
initializeTransformUtils(Registry); initializeTransformUtils(Registry);
initializeInstCombine(Registry); initializeInstCombine(Registry);
initializeTarget(Registry); initializeTarget(Registry);
initializeExpandLargeDivRemLegacyPassPass(Registry); initializeExpandLargeDivRemLegacyPassPass(Registry);
initializeExpandLargeFpConvertLegacyPassPass(Registry); initializeExpandLargeFpConvertLegacyPassPass(Registry);
initializeExpandMemCmpPassPass(Registry); initializeExpandMemCmpPassPass(Registry);
initializeScalarizeMaskedMemIntrinLegacyPassPass(Registry); initializeScalarizeMaskedMemIntrinLegacyPassPass(Registry);
initializeSelectOptimizePass(Registry); initializeSelectOptimizePass(Registry);
initializeCodeGenPreparePass(Registry); initializeCodeGenPreparePass(Registry);
initializeAtomicExpandPass(Registry); initializeAtomicExpandPass(Registry);
initializeRewriteSymbolsLegacyPassPass(Registry);
initializeWinEHPreparePass(Registry); initializeWinEHPreparePass(Registry);
initializeDwarfEHPrepareLegacyPassPass(Registry); initializeDwarfEHPrepareLegacyPassPass(Registry);
initializeSafeStackLegacyPassPass(Registry); initializeSafeStackLegacyPassPass(Registry);
initializeSjLjEHPreparePass(Registry); initializeSjLjEHPreparePass(Registry);
initializePreISelIntrinsicLoweringLegacyPassPass(Registry); initializePreISelIntrinsicLoweringLegacyPassPass(Registry);
initializeGlobalMergePass(Registry); initializeGlobalMergePass(Registry);
initializeIndirectBrExpandPassPass(Registry); initializeIndirectBrExpandPassPass(Registry);
initializeInterleavedLoadCombinePass(Registry); initializeInterleavedLoadCombinePass(Registry);
▲ Show 20 Lines • Show All 296 Lines • Show Last 20 Lines