From 9825ab04332a6b1b4692385ca86e715eb3f99190 Mon Sep 17 00:00:00 2001
From: Mehdi Amini <mehdi.amini@apple.com>
Date: Tue, 25 Oct 2016 00:48:48 +0000
Subject: Fix handling of %% format specifier in os_log builtins.

Returning `false` was stopping the parsing of further arguments,
which wasn't intended.

llvm-svn: 285047
---
 clang/lib/Analysis/OSLog.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'clang/lib/Analysis/OSLog.cpp')

diff --git a/clang/lib/Analysis/OSLog.cpp b/clang/lib/Analysis/OSLog.cpp
index b57e0d1d13f..04183883990 100644
--- a/clang/lib/Analysis/OSLog.cpp
+++ b/clang/lib/Analysis/OSLog.cpp
@@ -57,7 +57,7 @@ public:
     if (!FS.consumesDataArgument() &&
         FS.getConversionSpecifier().getKind() !=
             clang::analyze_format_string::ConversionSpecifier::PrintErrno)
-      return false;
+      return true;
 
     ArgsData.emplace_back();
     unsigned ArgIndex = FS.getArgIndex();
-- 
cgit v1.2.3