[PATCH] D83871: [llvm-readobj/readelf] - Refactor how the code dumps relocations.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 00:20:58 PDT 2020


MaskRay added a comment.

LLD uses `template RelTy` so there is prior art... For llvm-readobj, this seems to fix a misnomer. This is a +1 to me.

Have you considered https://bugs.llvm.org/show_bug.cgi?id=44257 Does this change make that easier or more difficult to fix?


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

https://reviews.llvm.org/D83871





More information about the llvm-commits mailing list