This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: patch to fix PR58967
- From: Robert Dewar <dewar at adacore dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Cc: David Edelsohn <dje dot gcc at gmail dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>
- Date: Mon, 04 Nov 2013 14:50:53 -0500
- Subject: Re: RFA: patch to fix PR58967
- Authentication-results: sourceware.org; auth=none
- References: <5277F436 dot 6040601 at redhat dot com>
On 11/4/2013 2:23 PM, Vladimir Makarov wrote:
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58967
The removed code is too old. To be honest, I even don't remember why I
added this. LRA has been changed a lot since this change and now it
works fine without it.
Whenever I see a comment like this, it reminds me to remind everyone
to comment your code! Do not assume you will remember why you wrote
what you wrote, so even if it is you who will look at your code, write
comments for yourself assuming you have totally forgotten!