[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