[PATCH] D101100: [ConstFold] Simplify a load's GEP operand through local aliases

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

Should we go forward with something in the direction of D101103 <https://reviews.llvm.org/D101103> instead, or is this still the way forward?



================
Comment at: llvm/lib/IR/Value.cpp:624-625
       V = cast<GlobalAlias>(V)->getAliasee();
+    } else if (StripKind == PSK_ForLoadOperand && isa<GlobalAlias>(V)) {
+      V = cast<GlobalAlias>(V)->getAliasee();
     } else if (StripKind == PSK_ForAliasAnalysis && isa<PHINode>(V) &&
----------------
Do we need checks for non-interposability on the alias and aliasee here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101100