[PATCH] D25184: Don't drop the llvm. prefix when renaming

[Rafael Espíndola via llvm-commits]

On 3 October 2016 at 07:01, Rafael Espíndola <rafael.espindola at gmail.com> wrote:
> On 3 October 2016 at 06:58, Artur Pilipenko <apilipenko at azulsystems.com> wrote:
>> apilipenko added a comment.
>>
>> The change seems fine, but I'm curious how we end up with an old renamed intrinsic declaration? For some reason I can't reproduce the problem locally.
>
> If you revert just the change in lib, don't you get a crash in the testcase?


And yes, BitcodeReader::materializeModule is never called.

Cheers,
Rafael