[PATCH] D104806: [ScalarEvolution] Make getMinusSCEV() fail for unrelated pointers.

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In D104806#2839319 <https://reviews.llvm.org/D104806#2839319>, @efriedma wrote:

> @reames didn't think the new API was necessary.  I'm sort of on the fence; I don't think adding a new API that's essentially the same thing as getMinusSCEV() is really that useful, but it is easier to audit.

I'd still prefer not adding a new API.  There aren't that many uses of this API.  I also don't really care though.  So I'll defer to Eli's preference here since he's the one actually doing the work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104806