This is an archive of the discontinued LLVM Phabricator instance.

[AA] Updates for D95543.
ClosedPublic

Authored by dfukalov on Apr 14 2021, 1:10 AM.

Details

Summary

Addressing latter comments in D95543:

  • AliasResult::Result renamed to AliasResult::Kind
  • Offset printing added for PartialAlias case in -aa-eval
  • Removed VisitedPhiBBs check from BasicAA'

Diff Detail

Event Timeline

dfukalov created this revision.Apr 14 2021, 1:10 AM
dfukalov requested review of this revision.Apr 14 2021, 1:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2021, 1:10 AM
dfukalov edited the summary of this revision. (Show Details)Apr 14 2021, 1:12 AM
nikic added inline comments.Apr 14 2021, 1:13 AM
llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
56

Why not AR.swap(true);?

dfukalov updated this revision to Diff 337407.Apr 14 2021, 4:45 AM

Addressing comment: using AliasResult::swap().

dfukalov marked an inline comment as done.Apr 14 2021, 4:48 AM
dfukalov added inline comments.
llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
56

Thanks, seems it was my mental fog )

nikic accepted this revision.Apr 14 2021, 11:40 AM

LGTM, thanks!

llvm/lib/Analysis/AliasAnalysisEvaluator.cpp
57

Might make sense to make true the default.

This revision is now accepted and ready to land.Apr 14 2021, 11:40 AM
This revision was automatically updated to reflect the committed changes.
dfukalov marked an inline comment as done.