[PATCH] D99414: [clang][tooling] Create SourceManager for DiagnosticsEngine before command-line parsing

Jordan Rupprecht via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 29 12:20:19 PDT 2021


rupprecht accepted this revision.
rupprecht added a comment.
This revision is now accepted and ready to land.

Thanks, the error message is much better now!

When applying this patch locally and also undoing the fix in our internal tool (i.e. change it to `-ferror-limit=-1`), the error I get is:

  message: "invalid integral value \'-1\' in \'-ferror-limit -1\'"

which, you know, actually tells me what to fix instead of giving some weird internal assertion :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99414



More information about the cfe-commits mailing list