[PATCH] D99546: [llvm][ValueMap] Share the value map between MappingContext

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

I agree abandoning the change makes sense, but it would be great if we could somehow collectively understand (and document) why we need a second ValueMap for comdats...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99546