[PATCH] D73131: [IR] Value::getPointerAlignment(): handle pointer constants

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D73131#1861781 <https://reviews.llvm.org/D73131#1861781>, @jdoerfert wrote:

> In D73131#1861624 <https://reviews.llvm.org/D73131#1861624>, @mlychkov wrote:
>
> > In D73131#1861595 <https://reviews.llvm.org/D73131#1861595>, @mlychkov wrote:
> >
> > > ...
> > >  Is this alignment accounting intended to be used for getelementptr instructions too?
> > >  If yes then may we cleanup extra user after we get TrailingZeros value?
> >
> >
> > Or is there a way to perform this computation without const_cast?
>
>
> The problem is `ConstantExpr::getPtrToInt`. Do we striclty need that?


That is the workhorse here, we can't get around using it.

Without seeing the code where this is causing trouble i'm somewhat unsure
on how this is a problem, or how to test that the fix would actually fix it.
Perhaps, after computing `TrailingZeros`, we need to explicitly DCE `CstInt`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73131