This is an archive of the discontinued LLVM Phabricator instance.

[Attributor] Disable the MustBeExecutedContext by default
AcceptedPublic

Authored by jdoerfert on Sep 9 2020, 10:17 PM.

Details

Summary

The MustBeExecutedContext is powerful but as of now we have no good way
to limit the creation of new AAs when traversing it. In order to get
compile time down we will disable the use of the MustBeExecutedContext
for now and revisit this once we have more control on the compile time
spend.

Tests are updated to run with the MBEC except MBEC.ll which runs with
and without.

Diff Detail

Event Timeline

jdoerfert created this revision.Sep 9 2020, 10:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2020, 10:18 PM
jdoerfert requested review of this revision.Sep 9 2020, 10:18 PM
sstefan1 accepted this revision.Sep 9 2020, 11:50 PM

LGTM!

Looks like parallel_deletion.ll needs to be updated as well.

This revision is now accepted and ready to land.Sep 9 2020, 11:50 PM