[PATCH] D38613: [ELF] - Change how we deal with unused synthetic sections.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 6 22:39:24 PDT 2017


davide added a comment.

I'm not sure how I feel about this.
It's true that reduces the number of lines to implement part of this feature.
OTOH, it scatters details that were in a single place into multiple (three, to be precise).
In some sense,it makes following this code harder to read.


https://reviews.llvm.org/D38613





More information about the llvm-commits mailing list