Improve the test
Shouldn't the new code fail with LLVM_ENABLE_ABI_BREAKING_CHECKS because of removing this line?
llvm::object::SectionRef::getName(StringRef) still returns error_code... ( it has 95 references so i didn't fix it to use Error instead)
Oh sorry, it should be Expected<StringRef>. Anyway, I haven't started the refactoring yet..
LGTM (assuming the output is consistent with GNU nm).
Ah, I see.
LGTM too. I'll update the doc patch.