<div dir="ltr">Ah, thanks. (I'm going to see about renaming/restructuring that API - to return Optional<APSInt> and be called 'get' rather than 'is' to make this more clear)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 5:04 AM Mikhail Maltsev <<a href="mailto:Mikhail.Maltsev@arm.com" target="_blank">Mikhail.Maltsev@arm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Yes, it does. isIntegerConstantExpr assigns a value to CoprocNoAP.<br>
<br>
--<br>
Regards,<br>
   Mikhail Maltsev<br>
<br>
________________________________________<br>
From: David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>><br>
Sent: Sunday, March 22, 2020 03:32<br>
To: Mikhail Maltsev; Mikhail Maltsev<br>
Cc: cfe-commits<br>
Subject: Re: [clang] 43606ef - Suppress an "unused variable" warning in release build<br>
<br>
Does "isIntegerConstantExpr" have side effects that are desired/necessary? Otherwise please change this to roll the isIntegerConstantExpr into the assert (so that it is only executed when asserts are enabled)<br>
<br>
On Tue, Mar 10, 2020 at 10:11 AM Mikhail Maltsev via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><mailto:<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>>> wrote:<br>
<br>
Author: Mikhail Maltsev<br>
Date: 2020-03-10T17:10:52Z<br>
New Revision: 43606efb6847fc9c79e7d93760a2a6191e8a8539<br>
<br>
URL: <a href="https://github.com/llvm/llvm-project/commit/43606efb6847fc9c79e7d93760a2a6191e8a8539" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/43606efb6847fc9c79e7d93760a2a6191e8a8539</a><br>
DIFF: <a href="https://github.com/llvm/llvm-project/commit/43606efb6847fc9c79e7d93760a2a6191e8a8539.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/43606efb6847fc9c79e7d93760a2a6191e8a8539.diff</a><br>
<br>
LOG: Suppress an "unused variable" warning in release build<br>
<br>
Added:<br>
<br>
<br>
Modified:<br>
    clang/lib/Sema/SemaChecking.cpp<br>
<br>
Removed:<br>
<br>
<br>
<br>
################################################################################<br>
diff  --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp<br>
index 24d0d9209a1d..8a2b4b019663 100644<br>
--- a/clang/lib/Sema/SemaChecking.cpp<br>
+++ b/clang/lib/Sema/SemaChecking.cpp<br>
@@ -2094,6 +2094,7 @@ bool Sema::CheckARMCoprocessorImmediate(const Expr *CoprocArg, bool WantCDE) {<br>
<br>
   llvm::APSInt CoprocNoAP;<br>
   bool IsICE = CoprocArg->isIntegerConstantExpr(CoprocNoAP, Context);<br>
+  (void)IsICE;<br>
   assert(IsICE && "Coprocossor immediate is not a constant expression");<br>
   int64_t CoprocNo = CoprocNoAP.getExtValue();<br>
   assert(CoprocNo >= 0 && "Coprocessor immediate must be non-negative");<br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><mailto:<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>