[PATCH] D115927: [AA] Correctly maintain the sign of PartiaAlias offset

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 17 06:49:43 PST 2021


dfukalov added inline comments.


================
Comment at: llvm/unittests/Analysis/AliasAnalysisTest.cpp:364-366
+  AR = AA.alias(Loc2, Loc1);
+  EXPECT_EQ(AR, AliasResult::PartialAlias);
+  EXPECT_EQ(-1, AR.getOffset());
----------------
Thanks!
Nit: I guess you can just extend the previous test "PartialAliasOffset" with this lines (and changed "-1" to "-4").


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

https://reviews.llvm.org/D115927



More information about the llvm-commits mailing list