From: Ian Lance Taylor <iant@google.com>
To: Mikolaj Zalewski <mikolajz@google.com>
Cc: binutils@sourceware.org
Subject: Re: [PATCH] gold: add cast to gold_unreachable to workaround gcc giving invalid "no return statement" warnings
Date: Fri, 14 Aug 2009 09:36:00 -0000 [thread overview]
Message-ID: <m38whm7n60.fsf@google.com> (raw)
In-Reply-To: <7ebec9e70908140228y5b15b376lcb4888aaddc3c2a3@mail.gmail.com> (Mikolaj Zalewski's message of "Fri\, 14 Aug 2009 11\:28\:13 +0200")
Mikolaj Zalewski <mikolajz@google.com> writes:
> Yes, this patch works.
Thanks for checking.
Committed.
Ian
> On Fri, Aug 14, 2009 at 10:34 AM, Ian Lance Taylor<iant@google.com> wrote:
>> Mikolaj Zalewski <mikolajz@google.com> writes:
>>
>>> I couldn't compile gold today because of the "no return statement"
>>> warning when using gold_unreachable. At
>>> http://gcc.gnu.org/ml/gcc-bugs/2007-11/msg01605.html I've found a
>>> workaround that should work for gcc >= 3.4. Such a patch that affects
>>> only gold_unreachable is ok?
>>>
>>> 2009-08-13 Mikolaj Zalewski <mikolajz@google.com>
>>>
>>> * gold.h (gold_unreachable): Add a cast.
>>
>> I would prefer a patch like this one, which gives us a way to clean it
>> up a few years down the road. Could you check whether this solves the
>> problem for you? If it does, I'll commit it. Thanks.
>>
>> Ian
>>
>>
>> 2009-08-14 Ian Lance Taylor <iant@google.com>
>>
>> * gold.h (FUNCTION_NAME): Define.
>> (gold_unreachable): Use FUNCTION_NAME.
>>
>>
>>
next prev parent reply other threads:[~2009-08-14 9:36 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-13 15:09 Mikolaj Zalewski
2009-08-14 8:34 ` Ian Lance Taylor
2009-08-14 9:28 ` Mikolaj Zalewski
2009-08-14 9:36 ` Ian Lance Taylor [this message]
2009-08-31 21:49 ` [PATCH] Gold: Added R_ARM_ABS8 relocation Viktor Kutuzov
2009-08-31 23:52 ` Gold: Testsuite Viktor Kutuzov
2009-09-01 0:46 ` Ian Lance Taylor
2009-09-01 1:02 ` Viktor Kutuzov
2009-09-01 2:10 ` Ian Lance Taylor
[not found] ` <6AE1604EE3EC5F4296C096518C6B77EEE56FBF60@mail.accesssoftek.com>
2009-09-02 1:31 ` Viktor Kutuzov
2009-09-01 0:56 ` [PATCH] Gold: Added R_ARM_ABS8 relocation Ian Lance Taylor
2009-09-01 19:53 ` [PATCH Take 2] " Viktor Kutuzov
2009-09-01 21:24 ` Ian Lance Taylor
2009-09-02 1:30 ` [PATCH] Gold: Added R_ARM_ABS8 relocation unit test Viktor Kutuzov
2009-09-16 0:23 ` [PATCH] Gold: Added R_ARM_GOT_PREL relocation and unit tests Viktor Kutuzov
2009-09-16 1:00 ` [Gold] Heads up. Working on interworking Viktor Kutuzov
2009-10-06 21:46 ` [GOLD] Heads up. Gold for mingw Viktor Kutuzov
2009-10-06 22:02 ` Vincent R.
2009-10-06 22:26 ` Viktor Kutuzov
2009-10-06 22:06 ` Matt Rice
2009-10-07 15:31 ` [PATCH] Gold: Added R_ARM_GOT_PREL relocation and unit tests Ian Lance Taylor
[not found] ` <6AE1604EE3EC5F4296C096518C6B77EEE56FBF62@mail.accesssoftek.com>
2009-09-03 23:26 ` [PATCH, Take 2] Gold: Added R_ARM_ABS8 relocation unit test Viktor Kutuzov
2009-09-27 7:35 ` Ian Lance Taylor
2009-09-29 23:30 ` Viktor Kutuzov
2009-09-29 23:40 ` Ian Lance Taylor
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m38whm7n60.fsf@google.com \
--to=iant@google.com \
--cc=binutils@sourceware.org \
--cc=mikolajz@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).