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

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71620