[PATCH] D67157: [lib/ObjectYAML] - Cleanup the private interface of ELFState<ELFT>. NFCI.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 01:43:25 PDT 2019


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

> I do not know why, but two methods accepted ELFState<ELFT> &State as

an argument, though they are not static. There is no visible reason for having that.

D62809 <https://reviews.llvm.org/D62809> accidentally added `ELFState<ELFT> &State` as the first parameter to two methods. This patch removes them.


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

https://reviews.llvm.org/D67157





More information about the llvm-commits mailing list