[PATCH] D64768: [InstructionSimplify] Apply sext/trunc after pointer stripping

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

This is fine with me. Though, I'm unsure if this should not live in the `stripAndAccumulateConstantOffsets` method.

The problem is that we have two bit-widths now, pointer before and after. Since we have to express offset in one of them, we have a choice.
I think we already guarantee that the offset is representable in the initial bit-width so that was what I choose. Question is, do we want
to change it to be in the resulting bit-width or not. Either way, we should add a comment to the stripAndAccumulateConstantOffsets method
explaining the situation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64768