This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [C++ Patch] PR 63265

From: Paolo Carlini <paolo dot carlini at oracle dot com>
To: Jason Merrill <jason at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 11 Nov 2014 15:58:05 +0100
Subject: Re: [C++ Patch] PR 63265
Authentication-results: sourceware.org; auth=none
References: <5460E8B1 dot 1030405 at oracle dot com> <5460F2E0 dot 5040801 at redhat dot com> <5462096E dot 9090308 at oracle dot com> <54620CD2 dot 7090304 at redhat dot com>

Hi,

On 11/11/2014 02:19 PM, Jason Merrill wrote:

On 11/11/2014 08:04 AM, Paolo Carlini wrote:
-    tree cond = RECUR (TREE_OPERAND (t, 0));
+    tree cond
+      = maybe_constant_value (fold_non_dependent_expr_sfinae
+                  (RECUR (TREE_OPERAND (t, 0)), tf_none));
I like this approach, but if the result of maybe_constant_valuedoesn't turn out to be an INTEGER_CST, we want to end up with theresult of RECUR rather than the result of fold_non_dependent_expr, asthe latter might not be suitable for subsequent tsubsting.

I see. Something like the below, then?

Thanks,
Paolo.

/////////////////

Attachment: patch_63265_3
Description: Text document

Follow-Ups:
- Re: [C++ Patch] PR 63265
  - From: Jason Merrill

References:
- [C++ Patch] PR 63265
  - From: Paolo Carlini
- Re: [C++ Patch] PR 63265
  - From: Jason Merrill
- Re: [C++ Patch] PR 63265
  - From: Paolo Carlini
- Re: [C++ Patch] PR 63265
  - From: Jason Merrill

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]