[PATCH] D66186: [Sema] Don't warn on printf('%hd', [char]) (PR41467)

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 22 23:37:33 PDT 2019


lebedev.ri added a comment.

Looks about right



================
Comment at: clang/lib/Sema/SemaChecking.cpp:8105-8106
+        if (ImplicitMatch == analyze_printf::ArgType::Match)
           return true;
+        else if (ImplicitMatch == analyze_printf::ArgType::NoMatchPedantic)
+          Pedantic = true;
----------------
No else after return.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66186





More information about the cfe-commits mailing list