[PATCH] D24088: Create a getelementptr instead of sub expr for ValueOffsetPair if the value is a pointer

[Sanjoy Das via llvm-commits]

sanjoy added a comment.

> What's your idea about writting a C++ testcase?


Sorry, fat fingered the previous response.  By a C++ test case, I meant writing some C++ code that makes the right calls to `getSCEV` and `SCEVExpander::expand` to trigger the bad behavior.  I'm not sure if that will be easy or even possible, but it is worth a try.


Repository:
  rL LLVM

https://reviews.llvm.org/D24088