[PATCH] D25936: Fix format string for err_os_log_argument_to_big (currently unused)

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 25 05:52:36 PDT 2016


sammccall added a comment.

In https://reviews.llvm.org/D25936#578341, @mehdi_amini wrote:

> Good catch! How did you find this?


We've got a library that parses diagnostics that consumes DiagnosticSemaKinds.inc, and it failed to parse this message format.

> We need a test though.
>  (I can take over if you want, as I introduced the issue in the first place)

Thanks! I'm pretty new to llvm and wasn't sure how to test this.
Looks like bkramer came up with one though.


Repository:
  rL LLVM

https://reviews.llvm.org/D25936





More information about the cfe-commits mailing list