[PATCH] D71620: [Attributor] AAValueConstantRange: Value range analysis using constant range

[Hideto Ueno via Phabricator via llvm-commits]

uenoku reopened this revision.
uenoku added a comment.
This revision is now accepted and ready to land.



In D71620#1803466 <https://reviews.llvm.org/D71620#1803466>, @jdoerfert wrote:

> In D71620#1802493 <https://reviews.llvm.org/D71620#1802493>, @uenoku wrote:
>
> > It seems the failure occurs when old pm is used. 
> >  I couldn't find what is causing this error.
> >
> > @jdoerfert Could you take a look at this?
>
>
>
>
> > opt: /b/s/w/ir/k/llvm-project/llvm/lib/Transforms/IPO/Attributor.cpp:6475: llvm::ChangeStatus llvm::Attributor::rewriteFunctionSignatures(): Assertion `OldFn->getNumUses() == 0 && "Unexpected leftover uses!"' failed.
>
> Should have been fixed by rGd2d2fb19f7ea <https://reviews.llvm.org/rGd2d2fb19f7ea3b192247ca2d6995e66229366cec>.
>
> > Called function must be a pointer!   call addrspace(1792) void <badref>(i32* nofree readnone undef) #10
>
> I haven't seen this running the test suite. I'll apply your patch and take a look.


Thank you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71620