On 03/08/2017 08:29 AM, Mark Wielaard wrote:
On Wed, 2017-03-08 at 08:22 -0500, Nathan Sidwell wrote:
https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00089.html?

Patch looks good to me (not a libiberty maintainer)

One question:
+  if (dc == NULL || dc->d_printing > 1)

Can dc ever be null here, or is this just extra defensiveness?

Yes, dc can be NULL. There is a similar check (and setting of
d_print_error) in d_print_comp_inner.

thanks. in case you'd not noticed Ian T's okayed my review. so this is good to go

nathan

--
Nathan Sidwell

Reply via email to