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

[Artur Pilipenko via llvm-commits]

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 we materialize the whole module we must remove old declarations from the module (see BitcodeReader::materializeModule). Althought if we use BitcodeReader::materialize old declaration will be kept in the module.


Repository:
  rL LLVM

https://reviews.llvm.org/D25184