[all-commits] [llvm/llvm-project] e92a8e: [SCEV] BuildConstantFromSCEV(): actually properly ...

[Roman Lebedev via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e92a8e0c743f83552fac37ecf21e625ba3a4b11e
      https://github.com/llvm/llvm-project/commit/e92a8e0c743f83552fac37ecf21e625ba3a4b11e
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2020-10-13 (Tue, 13 Oct 2020)

  Changed paths:
    M llvm/lib/Analysis/ScalarEvolution.cpp

  Log Message:
  -----------
  [SCEV] BuildConstantFromSCEV(): actually properly handle SExt-of-pointer case

As being pointed out by @efriedma in
https://reviews.llvm.org/rGaaafe350bb65#inline-4883
of course we can't just call ptrtoint in sign-extending case
and be done with it, because it will zero-extend.

I'm not sure what i was thinking there.

This is very much not an NFC, however looking at the user of
BuildConstantFromSCEV() i'm not sure how to actually show that
it results in a different constant expression.