This is a NFC patch moving the GNUELFDumper<ELFT>::printEnum() function from ELFDumper into ScopedPrinter.h for reuse.
jhenderson shchenz Higuoxing jsji
- Group Reviewers
- rG62c74d496bb4: [NFC] move GNUELFDumper::printEnum() into a common header for reuse.
Looks good! I originally wondered about this, as the behaviour doesn't seem really related to ScopedPrinter, but the EnumEntry class is defined in the same function, so I think this is the right place for it. Could you take a look at D114684 and see if there are any additional unit tests that could be added whilst you're doing this? (I'm thinking it would be good to add some, but could be a separate patch, depending on the order in which that and this patch land).