[PATCH] D132931: [Attributor] Simplify offset calculation for a constant GEP

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:1292
+        bool CannotFail = GEP->accumulateConstantOffset(DL, GEPOffset);
+        assert(CannotFail);
+        (void)CannotFail;
----------------
For scalable vectors this can still fail even if all indices are constant. Not sure if relevant in this context.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132931