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

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 01:45:58 PDT 2019


grimar added a comment.

In D67157#1658940 <https://reviews.llvm.org/D67157#1658940>, @MaskRay wrote:

> > 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.


Ah, it was me... :) Fine then.


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

https://reviews.llvm.org/D67157





More information about the llvm-commits mailing list