[lld] Remove shadowing "size" field from classes that inherit from SyntheticSection (PR #166323)

Peter Smith via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 17 02:52:23 PST 2025


smithp35 wrote:

Change LGTM.

>From looking at the history there was a series of refactorings in this area, where uncompressedSize was renamed to size field went from being a protected: `int64_t` for use to public `uint64_t` in 2022. For example: https://github.com/llvm/llvm-project/commit/53b674ee159ae0dd9f5e7492e1c0af45d2305e73 , https://github.com/llvm/llvm-project/commit/565b3b01b11f1e76bb6f93fe0da02309354525b5 and https://github.com/llvm/llvm-project/commit/e8fafafe8ef5276879d654639a011e02810697e4 

The use of size fields in the SyntheticSections will have predated that refactoring.


https://github.com/llvm/llvm-project/pull/166323


More information about the llvm-commits mailing list