[PATCH] D31028: [ELF] - Detemplate PltSection<ELFT> section.

[George Rimar via Phabricator via llvm-commits]

grimar added a comment.

In https://reviews.llvm.org/D31028#703875, @ruiu wrote:

> So you submitted a modified version of this patch as r298065? As a common practice, once you sent a patch for review, you are supposed to wait for an LGTM (if you think that a new patch is obvious, you can submit it to let other people do post-commit review, but it needs to be actually obvious and shouldn't be waiting for an LGTM). I appreciate your recent cleanup commits, but you want to slow down a bit here to follow the rule.


Not modified version of this patch, but a patch that changes nothing and just removes templating, leaving the single method templated. It is obvious trivial change I think. 
And I kept this patch on review because think it is better way and can be committed on top.

Do you think https://reviews.llvm.org/rL298065 really needed pre-commit review ?


https://reviews.llvm.org/D31028