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

[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 (expr));
+
+  if (EXCEPTIONAL_CLASS_P (expr))
+    return expr;

This seems like it could use a comment about why we don't actually wrap 
BASELINK as suggested by the previous comment.

OK with that change.

Jason