[PATCH] D152456: SeparateConstOffsetFromGEP: Preserve SCEV

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D152456#4440452 <https://reviews.llvm.org/D152456#4440452>, @arsenm wrote:

> In D152456#4410228 <https://reviews.llvm.org/D152456#4410228>, @nikic wrote:
>
>> Might be better to do that and not request SCEV at all?
>
> Possibly, but should be a separate patch. I'll file an issue to stop using SCEV here

Went to describe the issue in the ticket and ended up with the patch which seems to work


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

https://reviews.llvm.org/D152456