This is an archive of the discontinued LLVM Phabricator instance.

Differential D85381

[NewPM][opt] Add more codegen passes
ClosedPublic

Authored by aeubanks on Aug 5 2020, 5:19 PM.

Details

Reviewers
ychen
asbirlea
Commits
rGada4ee7cb8ff: [NewPM][opt] Add more codegen passes
Summary

Reduces number of failures by 92.

Diff Detail

Event Timeline

aeubanks created this revision.Aug 5 2020, 5:19 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 5 2020, 5:19 PM
Herald added subscribers: llvm-commits, jfb. · View Herald Transcript
aeubanks requested review of this revision.Aug 5 2020, 5:19 PM
Herald added a subscriber: aheejin. · View Herald TranscriptAug 5 2020, 5:19 PM
ychen accepted this revision.Aug 5 2020, 5:41 PM

LGTM. Thanks.

This revision is now accepted and ready to land.Aug 5 2020, 5:41 PM
Harbormaster completed remote builds in B67230: Diff 283452.Aug 5 2020, 5:55 PM
This revision was landed with ongoing or failed builds.Aug 5 2020, 7:00 PM
Closed by commit rGada4ee7cb8ff: [NewPM][opt] Add more codegen passes (authored by aeubanks). · Explain Why
This revision was automatically updated to reflect the committed changes.
aeubanks added a commit: rGada4ee7cb8ff: [NewPM][opt] Add more codegen passes.

Revision Contents

PathSize
llvm/
tools/
opt/
15 lines

Diff 283471

llvm/tools/opt/opt.cpp

Show First 20 Lines • Show All 485 Lines • ▼ Show 20 Linesstruct TimeTracerRAII {
} }
}; };
// For use in NPM transition. // For use in NPM transition.
// TODO: use a codegen version of PassRegistry.def/PassBuilder::is*Pass() once // TODO: use a codegen version of PassRegistry.def/PassBuilder::is*Pass() once
// it exists. // it exists.
static bool IsCodegenPass(StringRef Pass) { static bool IsCodegenPass(StringRef Pass) {
std::vector<StringRef> PassNamePrefix = { std::vector<StringRef> PassNamePrefix = {
"x86-", "xcore-", "wasm-", "systemz-", "ppc-", "nvvm-", "x86-", "xcore-", "wasm-", "systemz-", "ppc-", "nvvm-", "nvptx-",
"nvptx-", "mips-", "lanai-", "hexagon-", "bpf-", "avr-", "mips-", "lanai-", "hexagon-", "bpf-", "avr-", "thumb2-", "arm-",
"thumb2-", "arm-", "si-", "gcn-", "amdgpu-", "aarch64-"}; "si-", "gcn-", "amdgpu-", "aarch64-", "amdgcn-"};
std::vector<StringRef> PassNameContain = {"ehprepare"}; std::vector<StringRef> PassNameContain = {"ehprepare"};
std::vector<StringRef> PassNameExact = { std::vector<StringRef> PassNameExact = {
"safe-stack", "cost-model", "safe-stack", "cost-model",
"codegenprepare", "interleaved-load-combine", "codegenprepare", "interleaved-load-combine",
"unreachableblockelim", "sclaraized-masked-mem-intrin"}; "unreachableblockelim", "scalarize-masked-mem-intrin",
"verify-safepoint-ir", "divergence",
"infer-address-spaces", "atomic-expand",
"hardware-loops", "type-promotion",
"mve-tail-predication", "interleaved-access",
"global-merge", "pre-isel-intrinsic-lowering",
"expand-reductions", "indirectbr-expand",
"generic-to-nvvm", "expandmemcmp"};
for (const auto &P : PassNamePrefix) for (const auto &P : PassNamePrefix)
if (Pass.startswith(P)) if (Pass.startswith(P))
return true; return true;
for (const auto &P : PassNameContain) for (const auto &P : PassNameContain)
if (Pass.contains(P)) if (Pass.contains(P))
return true; return true;
for (const auto &P : PassNameExact) for (const auto &P : PassNameExact)
if (Pass == P) if (Pass == P)
▲ Show 20 Lines • Show All 533 Lines • Show Last 20 Lines