Page MenuHomePhabricator

AMDGPU: Disable some passes that are not meaningful
ClosedPublic

Authored by arsenm on Sep 21 2015, 10:04 PM.

Details

Reviewers
tstellarAMD
Summary

Don't run passes related to stack maps, garbage collection,
exceptions since these aren't useful for GPUs.

There might be a few more to turn off that I'm less sure about
(e.g. ShrinkWrapping) or I'm not sure how to disable
(SafeStack and StackProtector)

Diff Detail

Event Timeline

arsenm updated this revision to Diff 35342.Sep 21 2015, 10:04 PM
arsenm retitled this revision from to AMDGPU: Disable some passes that are not meaningful.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Sep 25 2015, 8:59 AM
tstellarAMD edited edge metadata.
tstellarAMD added inline comments.
lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
300–303

Would it be better to disable the passes from the GCNPassConfig constructor?

LGTM.

This revision is now accepted and ready to land.Sep 25 2015, 8:59 AM
arsenm closed this revision.Sep 25 2015, 10:43 AM

r248591

arsenm marked an inline comment as done.Sep 25 2015, 10:43 AM
arsenm added inline comments.
lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
300–303

I moved it to the AMDGPUPassConfig constructor