This improves the current llvm-rc parser by the ability of parsing ACCELERATORS statement.
|127 ↗||(On Diff #111734)|
Actually it looks like IntOrString has a constructor taking a Token, but currently isn't used anywhere and has no implementation. Is this intended and if so you could use this constructor instead of having if block?
Just the below style changes along with rnk@'s suggestion about ArrayRef<StringRef> and this is good.
|206–214 ↗||(On Diff #112442)|
if (!FlagResult->equals_lower(FlagDesc[FlagId])) continue;
|217–219 ↗||(On Diff #112442)|
std::string ExpectedList = llvm::join(FlagDesc, "/");