[PATCH] D42123: Derive GEP index type from Data Layout

David Chisnall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 03:36:58 PST 2018


theraven added a comment.

In https://reviews.llvm.org/D42123#987589, @delena wrote:

> I want to deprecate SCEVs for pointers if the index size is not equal to pointer size.
>  What do you think?


This will mean that you don't get a load of loop optimisations.  I think that's a pretty big hammer.  There's no reason why SCEV can't work here - we use it and have a bunch of patches against it to make it work in this context.  Please take a look at our code and see how much of it is applicable to you.


Repository:
  rL LLVM

https://reviews.llvm.org/D42123





More information about the llvm-commits mailing list