Re: [PATCH v2.4 of 02/14] Support for adding and stripping location_t wrapper nodes

2018-01-09 Thread Jason Merrill
On 01/09/2018 06:37 AM, David Malcolm wrote: + /* We should only be adding wrappers for constants and for decls, + or for some exceptional tree nodes (e.g. BASELINK in the C++ FE). */ + gcc_assert (CONSTANT_CLASS_P (expr) + || DECL_P (expr) + || EXCEPTIONAL_CLASS_P

[PATCH v2.4 of 02/14] Support for adding and stripping location_t wrapper nodes

2018-01-09 Thread David Malcolm
On Mon, 2018-01-08 at 19:08 +0100, Jakub Jelinek wrote: > On Mon, Jan 08, 2018 at 01:02:37PM -0500, David Malcolm wrote: > > Thanks Nathan and Jakub: a quick smoketest using TREE_LANG_FLAG_0 > > worked, and fixes this issue. > > > > However, should I be using a TREE_LANG_FLAG for something that's