[llvm-commits] Patch for bug 14672

Krzysztof Parzyszek kparzysz at codeaurora.org
Thu Dec 20 12:47:08 PST 2012


Could someone review this patch?

Thank you,
-Krzysztof


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
hosted by The Linux Foundation
-------------- next part --------------
>From 3d75dba0486d2f2fd68376d4c0a83e8733f33029 Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek <kparzysz at codeaurora.org>
Date: Thu, 20 Dec 2012 14:19:30 -0600
Subject: [PATCH] Add ${...} in tests of clang_arcmt and clang_static_analyzer
 in configure.ac.

---
 autoconf/configure.ac |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/autoconf/configure.ac b/autoconf/configure.ac
index 0395931..08e7f32 100644
--- a/autoconf/configure.ac
+++ b/autoconf/configure.ac
@@ -511,10 +511,10 @@ AC_ARG_ENABLE(clang-rewriter,
 case "$enableval" in
   yes) AC_SUBST(ENABLE_CLANG_REWRITER,[1]) ;;
   no)  
-    if test clang_arcmt != "no" ; then
+    if test ${clang_arcmt} != "no" ; then
       AC_MSG_ERROR([Cannot enable clang ARC Migration Tool while disabling rewriter.])
     fi
-    if test clang_static_analyzer != "no" ; then
+    if test ${clang_static_analyzer} != "no" ; then
       AC_MSG_ERROR([Cannot enable clang static analyzer while disabling rewriter.])
     fi
     AC_SUBST(ENABLE_CLANG_REWRITER,[0]) 
-- 
1.7.6.4



More information about the llvm-commits mailing list