[PATCH] D37038: Replace temp MD nodes with unique/distinct before cloning

[Paul Robinson via Phabricator via cfe-commits]

probinson added a comment.

In https://reviews.llvm.org/D37038#855559, @aprantl wrote:

> This may have gotten lost earlier:  Would it be possible to instruct CloneFunction to not clone any temporary MDNodes via one of the flags that are passed to the ValueMapper?


I did look at that, sorry for not reporting back.  ValueMapper seems to be quite adamant about having non-temporary nodes.  I can try pursuing that further but it would be quite the "learning experience" I think.
Alternatively we could try deferring the thunk cloning until after all other codegen, but that also seems like a pretty hacky approach.


https://reviews.llvm.org/D37038