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

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 11:04:28 PDT 2017

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.


More information about the cfe-commits mailing list