[PATCH] D96932: [SampleFDO] Support enabling -funique-internal-linkage-name

Wed Feb 24 10:57:29 PST 2021

wmi added a comment.

In D96932#2584061 <https://reviews.llvm.org/D96932#2584061>, @hoy wrote:

> BTW, in case of duplicated functions due to multi-versioning, the profile generator may not be able to merge them as expected without properly canonicalizing function names, the compiler might be able to achieve that though. As of now, it seems that the last profile for a given canonical name always supersedes all others in the compiler reader. This may need a change for a "merge" instead.

Right. Do context string need to be canonicalized? If it is needed, the current canonicalization needs some enhancement to support that.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96932/new/

https://reviews.llvm.org/D96932