This is an archive of the discontinued LLVM Phabricator instance.

[NewPM][opt] Fix -globals-aa not being recognized as alias analysis in NPM
ClosedPublic

Authored by aeubanks on Sep 14 2020, 6:40 PM.

Details

Summary

Was missing MODULE_ALIAS_ANALYSIS, previously only FUNCTION_ALIAS_ANALYSIS was taken into account.

Diff Detail

Event Timeline

aeubanks created this revision.Sep 14 2020, 6:40 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 14 2020, 6:40 PM
aeubanks requested review of this revision.Sep 14 2020, 6:40 PM
asbirlea accepted this revision.Sep 14 2020, 7:22 PM
asbirlea added inline comments.
llvm/test/Analysis/GlobalsModRef/comdat-ipo.ll
1

Add -enable-new-pm=0 on the first line?

This revision is now accepted and ready to land.Sep 14 2020, 7:22 PM
aeubanks updated this revision to Diff 291979.Sep 15 2020, 11:14 AM

add -enable-new-pm=0

This revision was landed with ongoing or failed builds.Sep 15 2020, 11:18 AM
This revision was automatically updated to reflect the committed changes.