Re: [C++ Patch] More accurate location for conditional expressions

[Jason Merrill]

On 05/21/2013 06:13 AM, Paolo Carlini wrote:
@@ -2141,7 +2141,8 @@ rationalize_conditional_expr (enum tree_code code,
        gcc_assert (!TREE_SIDE_EFFECTS (op0)
                  && !TREE_SIDE_EFFECTS (op1));
        return
-       build_conditional_expr (build_x_binary_op (input_location,
+       build_conditional_expr (input_location,
+                               build_x_binary_op (input_location,
                                                   (TREE_CODE (t) == MIN_EXPR
                                                    ? LE_EXPR : GE_EXPR),
                                                   op0, TREE_CODE (op0),
@@ -2154,7 +2155,7 @@ rationalize_conditional_expr (enum tree_code code,
      }
    return
-    build_conditional_expr (TREE_OPERAND (t, 0),
+    build_conditional_expr (input_location, TREE_OPERAND (t, 0),

In rationalize_conditional_expr, it seems to me that we should be using 
EXPR_LOC_OR_HERE (t) rather than input_location.  OK with that change.

Jason