[PATCH] D61846: [DAGCombiner] Fix invalid alias analysis.

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 01:19:33 PDT 2019


courbet marked 2 inline comments as done.
courbet added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:19802
   bool IsAlias;
   if (BaseIndexOffset::computeAliasing(Op0, MUC0.NumBytes, Op1, MUC1.NumBytes,
+                                       DAG, IsAlias))
----------------
echristo wrote:
> Can you comment this? Given it's already caused a problem and isn't obvious what's going on necessarily.
Sure. 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61846





More information about the llvm-commits mailing list