This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[C++ Patch] PR 81061 ("[7/8 Regression] ICE modifying read-only variable")

From: Paolo Carlini <paolo dot carlini at oracle dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: Nathan Sidwell <nathan at acm dot org>, Jason Merrill <jason at redhat dot com>
Date: Wed, 13 Dec 2017 11:32:53 +0100
Subject: [C++ Patch] PR 81061 ("[7/8 Regression] ICE modifying read-only variable")
Authentication-results: sourceware.org; auth=none

Hi,

in this simple error recovery regression we ICE during gimplificationafter sensible diagnostic about assigning to a read-only location. Theproblem can be avoided by simply returning immediately error_mark_nodeupon cxx_readonly_error - the rest of the function does the same, ie,doesn't try to proceed when complain & tf_error. I also noticed thatclang appears to behave in the same way for this error. Tested x86_64-linux.


Thanks, Paolo

////////////////////////

Attachment: CL_81061
Description: Text document

Attachment: patch_81061
Description: Text document

Follow-Ups:
- Re: [C++ Patch] PR 81061 ("[7/8 Regression] ICE modifying read-only variable")
  - From: Jason Merrill

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]