[PATCH] D127902: [Object][COFF] Improve section name parsing

[Pengxuan Zheng via Phabricator via llvm-commits]

pzheng added inline comments.


================
Comment at: llvm/lib/Object/COFFObjectFile.cpp:1159
     } else {
       if (Name.substr(1).consumeInteger(10, Offset))
         return createStringError(object_error::parse_failed,
----------------
rnk wrote:
> This can now be getAsInteger, right?
Compared to getAsInteger, consumeInteger is a little more error tolerant and can handle inputs like "/4abcdef". However, I haven't actually seen any input like that in the wild yet, so it's probably fine if we use getAsInteger. Thoughts?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127902/new/

https://reviews.llvm.org/D127902