On Thu, 2020-12-10 at 11:21 -0500, Patrick Palka via Gcc-patches wrote:
> +         if (entry->diagnose_instability)
> +           {
> +             error_at (EXPR_LOCATION (ATOMIC_CONSTR_EXPR (entry->atom)),
> +                       "satisfaction value of atomic constraint %qE changed "
> +                       "from %qE to %qE", entry->atom, entry->result, 
> result);
> +             inform (entry->location,
> +                     "satisfaction value first evaluated to %qE from here",
> +                     entry->result);
> +           }

I only glanced at the rest of the patch, but a minor nit I spotted
here: please add an
                auto_diagnostic_group d;
to this block to associate the "note" with the "error".


Dave

Reply via email to