[PATCH] D63521: Teach the symbolizer lib symbolize objects directly.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

In D63521#1562497 <https://reviews.llvm.org/D63521#1562497>, @ychen wrote:

> Hi @MaskRay, from @rnk 's inputs, seems this is not a common case worth pursuing at least for the moment, is it ok to move forward with the first choice I described?


My comment on your first version of this patch is about the potential code duplication. Now the codeview concern of the code duplication has been addressed.

The current version doesn't look bad to me. Do you think the initial version is better?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63521