[PATCH] D99121: [IR][InstCombine] IntToPtr Produces Typeless Pointer To Byte

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 24 20:56:25 PDT 2021


dblaikie added a comment.

In D99121#2649546 <https://reviews.llvm.org/D99121#2649546>, @ruiling wrote:

>> No, sorry I don't - I ran out of steam after the initial work, and haven't been able to get back into it. A few folks have picked up my slack in the last year or two & made some incremental progress.
>>
>> It'd be good to tag any workarounds somehow (I don't know how, exactly) to be sure they're cleaned up as things are sorted out.
>>
>> Long and the short of it: If these bugs matter to you, probably not worth waiting for the general fix (but more help would be appreciated if you wanted to work on the long term solution) & workarounds are probably reasonable.
>
> Sounds we still need a long way to get there? Do we think the patch acceptable as a short-term solution?

Guess it's a tradeoff. Doesn't look like a load of code/complexity. - but equally, does it provide much value? Or is it enough to know it can eventually be addressed in a more general manner?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99121



More information about the cfe-commits mailing list