[PATCH] D93747: Rename debug linkage name with -funique-internal-linkage-names

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 23 11:46:34 PST 2020


dblaikie added a comment.

Please remove the clang test change - if this is an LLVM change with LLVM test coverage, that's enough. (we don't generally test every LLVM change from both LLVM and Clang)

I'd still be curious if you could look around to see whether there are other cases of function splitting/cloning/etc to see how they deal with updating the DISubprograms, to see if there's some prior art that should be used here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93747



More information about the cfe-commits mailing list