public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* restore "do {" mistakenly taken out in  SUBSUBTARGET_OVERRIDE_OPTIONS  for ppc-vxworks
@ 2012-06-01 16:39 Olivier Hainque
  2012-06-01 17:12 ` Eric Botcazou
  0 siblings, 1 reply; 4+ messages in thread
From: Olivier Hainque @ 2012-06-01 16:39 UTC (permalink / raw)
  To: GCC Patches; +Cc: Olivier Hainque

[-- Attachment #1: Type: text/plain, Size: 241 bytes --]

This fixes a glitch introduced by me with rev 187581.
Committing as obvious.

2012-07-01  Olivier Hainque  <hainque@adacore.com>

	* config/rs6000/vxworks.h (SUBSUBTARGET_OVERRIDE_OPTIONS): Restore
	the "do {" part of the do-while(0) loop.


[-- Attachment #2: vxoverride.diff --]
[-- Type: application/octet-stream, Size: 387 bytes --]

Index: vxworks.h
===================================================================
--- vxworks.h	(revision 188113)
+++ vxworks.h	(working copy)
@@ -124,6 +124,7 @@
 
 #undef SUBSUBTARGET_OVERRIDE_OPTIONS
 #define SUBSUBTARGET_OVERRIDE_OPTIONS		\
+  do {						\
   if (!global_options_set.x_g_switch_value)	\
     g_switch_value = SDATA_DEFAULT_SIZE;	\
   VXWORKS_OVERRIDE_OPTIONS;			\

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: restore "do {" mistakenly taken out in  SUBSUBTARGET_OVERRIDE_OPTIONS  for ppc-vxworks
  2012-06-01 16:39 restore "do {" mistakenly taken out in SUBSUBTARGET_OVERRIDE_OPTIONS for ppc-vxworks Olivier Hainque
@ 2012-06-01 17:12 ` Eric Botcazou
  2012-06-02  5:59   ` Olivier Hainque
  0 siblings, 1 reply; 4+ messages in thread
From: Eric Botcazou @ 2012-06-01 17:12 UTC (permalink / raw)
  To: Olivier Hainque; +Cc: gcc-patches

> 2012-07-01  Olivier Hainque  <hainque@adacore.com>
>
> 	* config/rs6000/vxworks.h (SUBSUBTARGET_OVERRIDE_OPTIONS): Restore
> 	the "do {" part of the do-while(0) loop.

This is needed on the 4.7 branch as well.

-- 
Eric Botcazou

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: restore "do {" mistakenly taken out in  SUBSUBTARGET_OVERRIDE_OPTIONS  for ppc-vxworks
  2012-06-01 17:12 ` Eric Botcazou
@ 2012-06-02  5:59   ` Olivier Hainque
  2012-06-02  7:17     ` Eric Botcazou
  0 siblings, 1 reply; 4+ messages in thread
From: Olivier Hainque @ 2012-06-02  5:59 UTC (permalink / raw)
  To: Eric Botcazou; +Cc: Olivier Hainque, gcc-patches


On Jun 1, 2012, at 19:09 , Eric Botcazou wrote:

>> 	* config/rs6000/vxworks.h (SUBSUBTARGET_OVERRIDE_OPTIONS): Restore
>> 	the "do {" part of the do-while(0) loop.
> 
> This is needed on the 4.7 branch as well.

 Hmm, I don't think so. The removal was part of the E500
 related reorg, which was not committed to the branch.

 Indeed, the original code, with both the E500 resets and
 the 'do {', is there.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: restore "do {" mistakenly taken out in  SUBSUBTARGET_OVERRIDE_OPTIONS  for ppc-vxworks
  2012-06-02  5:59   ` Olivier Hainque
@ 2012-06-02  7:17     ` Eric Botcazou
  0 siblings, 0 replies; 4+ messages in thread
From: Eric Botcazou @ 2012-06-02  7:17 UTC (permalink / raw)
  To: Olivier Hainque; +Cc: gcc-patches

>  Hmm, I don't think so. The removal was part of the E500
>  related reorg, which was not committed to the branch.

You're right, I looked at the wrong branch.

-- 
Eric Botcazou

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-06-02  7:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-01 16:39 restore "do {" mistakenly taken out in SUBSUBTARGET_OVERRIDE_OPTIONS for ppc-vxworks Olivier Hainque
2012-06-01 17:12 ` Eric Botcazou
2012-06-02  5:59   ` Olivier Hainque
2012-06-02  7:17     ` Eric Botcazou

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).