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

2021-03-24 Thread Ruiling, Song via Phabricator via cfe-commits
ruiling added a comment. > 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

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

2021-03-25 Thread Ruiling, Song via Phabricator via cfe-commits
ruiling added a comment. >> 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 > eventuall

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

2021-03-25 Thread Ruiling, Song via Phabricator via cfe-commits
ruiling added a comment. Then what do you think of my initial change for the issue (D99051 ) considering it does not show regression in any existing test? @nikic @nlopes It has a test showing the initial problem I want to solve. @lebedev.ri helped writing this p