[PATCH] D124527: [GVN] Encode GEPs in offset representation

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/GVN.cpp:408
+  APInt ConstantOffset(BitWidth, 0);
+  if (PtrTy->isOpaquePointerTy() &&
+      GEP->collectOffset(DL, BitWidth, VariableOffsets, ConstantOffset)) {
----------------
why don't we do this for typed pointers as well?


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

https://reviews.llvm.org/D124527