On Thu, Dec 17, 2015 at 5:55 AM, Andrew Pinski <pins...@gmail.com> wrote:
> Hi,
>   PR 68948 shows a case which has been broken for a long time and very
> hard to see.  So I am recommending that we introduce an assert inside
> the gimplifier if we process an error_mark_node and there was no
> errors or sorrys from the front-end; I will send a patch if there is a
> general feeling that it is a good idea.
>
> Good idea or bad idea?  This will prevent cases like PR 68948 from
> happening and make it easier to understand why the wrong code is
> happening.

Where do we let error_mark_node through?  We should hit

           default:
              gcc_unreachable ();

in gimplify_expr.

> Thanks,
> Andrew

Reply via email to