[PATCH] D100454: [AA] Updates for D95543.

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov marked an inline comment as done.
dfukalov added inline comments.


================
Comment at: llvm/lib/Analysis/AliasAnalysisEvaluator.cpp:56
       std::swap(o1, o2);
+      // Change offset sign for the local AR, for printing only.
+      if (AR == AliasResult::PartialAlias && AR.hasOffset())
----------------
nikic wrote:
> Why not `AR.swap(true);`?
Thanks, seems it was my mental fog )


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100454/new/

https://reviews.llvm.org/D100454