Page MenuHomePhabricator

[Attributor][PM] Introduce `-attributor-enable={none,cgscc,module,all}`
ClosedPublic

Authored by jdoerfert on Apr 21 2020, 9:24 AM.

Details

Summary

The old command line option -attributor-disable was too coarse grained
as we want to measure the effects of the module or cgscc pass without
the other as well.

Since none is the default there is no real functional change.

Diff Detail

Event Timeline

jdoerfert created this revision.Apr 21 2020, 9:24 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2020, 9:25 AM
lebedev.ri accepted this revision.Apr 21 2020, 9:57 AM

Looks reasonable to me

llvm/include/llvm/Transforms/IPO/Attributor.h
2868–2869

1 << 0
1 << 1

This revision is now accepted and ready to land.Apr 21 2020, 9:57 AM
This revision was automatically updated to reflect the committed changes.