C++: Fix crash in warn_for_memset within templates (PR c++/83814)
[gcc.git] / gcc / cp / ChangeLog
1 2018-01-17 David Malcolm <dmalcolm@redhat.com>
2
3 PR c++/83814
4 * expr.c (fold_for_warn): Move from c-common.c, reducing to just
5 the C++ part. If processing a template, call
6 fold_non_dependent_expr rather than fully folding.
7
8 2018-01-17 Jason Merrill <jason@redhat.com>
9
10 PR c++/81067 - redundant NULL warning.
11 * call.c (convert_like_real): Restore null_node handling.
12
13 2018-01-17 Jason Merrill <jason@redhat.com>
14
15 PR c++/81843 - ICE with variadic member template.
16 PR c++/72801
17 * pt.c (unify_pack_expansion): Don't try to deduce enclosing
18 template args.
19
20 2018-01-17 David Malcolm <dmalcolm@redhat.com>
21
22 PR c++/83799
23 * pt.c (type_dependent_expression_p): Strip any location wrapper
24 before testing tree codes.
25 (selftest::test_type_dependent_expression_p): New function.
26 (selftest::cp_pt_c_tests): Call it.
27
28 2018-01-17 Nathan Sidwell <nathan@acm.org>
29
30 PR c++/83739
31 * pt.c (tsubst_expr) <case RANGE_FOR_STMT>: Rebuild a range_for if
32 this not a final instantiation.
33
34 2018-01-16 Jason Merrill <jason@redhat.com>
35
36 PR c++/83714 - ICE checking return in template.
37 * typeck.c (check_return_expr): Call build_non_dependent_expr.
38
39 2018-01-16 Jakub Jelinek <jakub@redhat.com>
40
41 PR c++/83817
42 * pt.c (tsubst_copy_and_build) <case CALL_EXPR>: If function
43 is AGGR_INIT_EXPR rather than CALL_EXPR, set AGGR_INIT_FROM_THUNK_P
44 instead of CALL_FROM_THUNK_P.
45
46 PR c++/83825
47 * name-lookup.c (member_vec_dedup): Return early if len is 0.
48 (resort_type_member_vec, set_class_bindings,
49 insert_late_enum_def_bindings): Use vec qsort method instead of
50 calling qsort directly.
51
52 2018-01-15 Martin Sebor <msebor@redhat.com>
53
54 PR c++/83588
55 * class.c (find_flexarrays): Make a record of multiple flexible array
56 members.
57
58 2018-01-12 Jason Merrill <jason@redhat.com>
59
60 PR c++/83186 - ICE with static_cast of list-initialized temporary.
61 * typeck.c (build_static_cast): Use build_non_dependent_expr.
62
63 2018-01-12 Nathan Sidwell <nathan@acm.org>
64
65 * cp-tree.h (mark_rvalue_use): Add parm name.
66 * expr.c (mark_lvalue_use, mark_lvalue_use_nonread): Move next to
67 mark_rvalue_use.
68 * call.c (convert_like_real): Fix formatting.
69
70 2018-01-11 Jason Merrill <jason@redhat.com>
71
72 PR c++/82728 - wrong -Wunused-but-set-variable
73 PR c++/82799
74 PR c++/83690
75 * call.c (perform_implicit_conversion_flags): Call mark_rvalue_use.
76 * decl.c (case_conversion): Likewise.
77 * semantics.c (finish_static_assert): Call
78 perform_implicit_conversion_flags.
79
80 2018-01-11 Nathan Sidwell <nathan@acm.org>
81
82 * method.c (enum mangling_flags): Delete long-dead enum.
83
84 2018-01-10 Paolo Carlini <paolo.carlini@oracle.com>
85
86 * parser.c (cp_parser_std_attribute_spec): When
87 token_pair::require_open / require_close return false simply
88 return error_mark_node, avoid duplicate cp_parser_error about
89 expected '(' / ')', respectively.
90
91 2018-01-10 David Malcolm <dmalcolm@redhat.com>
92
93 PR c++/43486
94 * call.c (null_ptr_cst_p): Strip location wrappers when
95 converting from '0' to a pointer type in C++11 onwards.
96 (conversion_null_warnings): Replace comparison with null_node with
97 call to null_node_p.
98 (build_over_call): Likewise.
99 * cp-gimplify.c (cp_fold): Remove the early bailout when
100 processing_template_decl.
101 * cp-lang.c (selftest::run_cp_tests): Call
102 selftest::cp_pt_c_tests and selftest::cp_tree_c_tests.
103 * cp-tree.h (cp_expr::maybe_add_location_wrapper): New method.
104 (selftest::run_cp_tests): Move decl to bottom of file.
105 (null_node_p): New inline function.
106 (selftest::cp_pt_c_tests): New decl.
107 (selftest::cp_tree_c_tests): New decl.
108 * cvt.c (build_expr_type_conversion): Replace comparison with
109 null_node with call to null_node_p.
110 * error.c (args_to_string): Likewise.
111 * except.c (build_throw): Likewise.
112 * mangle.c (write_expression): Skip location wrapper nodes.
113 * parser.c (literal_integer_zerop): New function.
114 (cp_parser_postfix_expression): Call maybe_add_location_wrapper on
115 the result for RID_TYPEID. Pass true for new "wrap_locations_p"
116 param of cp_parser_parenthesized_expression_list. When calling
117 warn_for_memset, replace integer_zerop calls with
118 literal_integer_zerop, eliminating the double logical negation
119 cast to bool. Eliminate the special-casing for CONST_DECL in
120 favor of the fold_for_warn within warn_for_memset.
121 (cp_parser_parenthesized_expression_list): Add "wrap_locations_p"
122 param, defaulting to false. Convert "expr" to a cp_expr, and call
123 maybe_add_location_wrapper on it when wrap_locations_p is true.
124 (cp_parser_unary_expression): Call maybe_add_location_wrapper on
125 the result for RID_ALIGNOF and RID_SIZEOF.
126 (cp_parser_builtin_offsetof): Likewise.
127 * pt.c: Include "selftest.h".
128 (tsubst_copy): Handle location wrappers.
129 (tsubst_copy_and_build): Likewise.
130 (build_non_dependent_expr): Likewise.
131 (selftest::test_build_non_dependent_expr): New function.
132 (selftest::cp_pt_c_tests): New function.
133 * tree.c: Include "selftest.h".
134 (lvalue_kind): Handle VIEW_CONVERT_EXPR location wrapper nodes.
135 (selftest::test_lvalue_kind): New function.
136 (selftest::cp_tree_c_tests): New function.
137 * typeck.c (string_conv_p): Strip any location wrapper from "exp".
138 (cp_build_binary_op): Replace comparison with null_node with call
139 to null_node_p.
140 (build_address): Use location of operand when building address
141 expression.
142
143 2018-01-10 Marek Polacek <polacek@redhat.com>
144
145 PR c++/82541
146 * call.c (build_conditional_expr_1): Check complain before warning.
147 * pt.c (tsubst_copy_and_build) <case COND_EXPR>: Suppress
148 -Wduplicated-branches.
149
150 2018-01-10 Jakub Jelinek <jakub@redhat.com>
151
152 PR c++/81327
153 * call.c (maybe_warn_class_memaccess): Add forward declaration.
154 Change last argument from tree * to const vec<tree, va_gc> *, adjust
155 args uses and check number of operands too. Don't strip away any
156 nops. Use maybe_constant_value when looking for INTEGER_CST args.
157 Deal with src argument not having pointer type. Check
158 tree_fits_uhwi_p before calling tree_to_uhwi. Remove useless
159 test.
160 (build_over_call): Call maybe_warn_class_memaccess here on the
161 original arguments.
162 (build_cxx_call): Rather than here on converted arguments.
163
164 2018-01-10 Paolo Carlini <paolo.carlini@oracle.com>
165
166 PR c++/81055
167 * init.c (build_vec_init): Avoid building an INIT_EXPR with
168 error_mark_node as second argument.
169
170 2018-01-09 Jakub Jelinek <jakub@redhat.com>
171
172 PR c++/83734
173 * constexpr.c (cxx_eval_statement_list): Ignore DEBUG_BEGIN_STMTs
174 in STATEMENT_LIST. Remove unneeded assert.
175
176 2018-01-03 Richard Sandiford <richard.sandiford@linaro.org>
177 Alan Hayward <alan.hayward@arm.com>
178 David Sherwood <david.sherwood@arm.com>
179
180 * constexpr.c (cxx_eval_array_reference): Handle polynomial
181 VECTOR_CST_NELTS.
182 (cxx_fold_indirect_ref): Handle polynomial TYPE_VECTOR_SUBPARTS.
183 * call.c (build_conditional_expr_1): Likewise.
184 * decl.c (cp_finish_decomp): Likewise.
185 * mangle.c (write_type): Likewise.
186 * typeck.c (structural_comptypes): Likewise.
187 (cp_build_binary_op): Likewise.
188 * typeck2.c (process_init_constructor_array): Likewise.
189
190 2018-01-03 Jakub Jelinek <jakub@redhat.com>
191
192 PR c++/83555
193 * typeck.c (build_static_cast_1): For static casts to reference types,
194 call build_base_path with flag_delete_null_pointer_checks as nonnull
195 instead of always false. When -fsanitize=null, call
196 ubsan_maybe_instrument_reference on the NULL reference INTEGER_CST.
197 * cp-gimplify.c (cp_genericize_r): Don't walk subtrees of UBSAN_NULL
198 call if the first argument is INTEGER_CST with REFERENCE_TYPE.
199
200 2018-01-03 Nathan Sidwell <nathan@acm.org>
201
202 PR c++/83667
203 * method.c (make_alias_for): Copy DECL_CONTEXT.
204
205 2018-01-03 Marek Polacek <polacek@redhat.com>
206
207 PR c++/83592
208 * decl.c (grokdeclarator): Don't warn about MVP in typename context.
209
210 2018-01-03 Jakub Jelinek <jakub@redhat.com>
211
212 PR preprocessor/83602
213 * name-lookup.c (lookup_name_fuzzy): Don't use macro_use_before_def
214 for builtin macros.
215
216 PR c++/83634
217 * cp-gimplify.c (cp_fold) <case NOP_EXPR>: If the operand folds to
218 error_mark_node, return error_mark_node.
219
220 Update copyright years.
221
222 2018-01-02 Jakub Jelinek <jakub@redhat.com>
223
224 PR c++/83556
225 * tree.c (replace_placeholders_r): Pass NULL as last argument to
226 cp_walk_tree instead of d->pset. If non-TREE_CONSTANT and
227 non-PLACEHOLDER_EXPR tree has been seen already, set *walk_subtrees
228 to false and return.
229 (replace_placeholders): Pass NULL instead of &pset as last argument
230 to cp_walk_tree.
231
232 2018-01-02 Nathan Sidwell <nathan@acm.org>
233
234 * constexpr.c (cxx_bind_parameters_in_call): Remove unneeded local
235 lval var.
236 \f
237 Copyright (C) 2018 Free Software Foundation, Inc.
238
239 Copying and distribution of this file, with or without modification,
240 are permitted in any medium without royalty provided the copyright
241 notice and this notice are preserved.