Re: [PATCH v2 0/4] some vxworks/powerpc patches

2018-08-03 Thread Olivier Hainque


Hello Rasmus,

(Back after a summer break & some backlog preemption)

Thanks for resubmitting those patches. 

Re reviewing; will get back to you when I'm done,
hopefully soon :)

Olivier

> On 28 Jun 2018, at 10:43, Rasmus Villemoes  wrote:
> 
> Patches 1, 2 and 4 (the latter was number 6 previously) are unchanged
> from the first round, apart from small changes in the commit log
> wording. Patch 3 now includes parentheses in the macro
> definition. Patches 4 and 5 are dropped.
> 
> 4/4 is mostly a minor optimization, omitting a tiny bit of dead code
> from the compiler binary. But the preprocessor directives also serve
> as a reminder that the custom vxworks functions will not be used if
> the compiler is built with --enable-initfini-array.
> 
> Rasmus Villemoes (4):
>  vxworks: add target/h/wrn/coreip to the set of system include paths
>  libgcc: add crt{begin,end} for powerpc-wrs-vxworks target
>  vxworks: enable use of .init_array/.fini_array for cdtors
>  vxworks: don't define vxworks_asm_out_constructor when using
>.init_array
> 
> gcc/config/vxworks.c |  9 +++--
> gcc/config/vxworks.h | 21 +++--
> libgcc/config.host   |  1 +
> 3 files changed, 23 insertions(+), 8 deletions(-)
> 
> -- 
> 2.16.4
> 



[PATCH v2 0/4] some vxworks/powerpc patches

2018-06-28 Thread Rasmus Villemoes
Patches 1, 2 and 4 (the latter was number 6 previously) are unchanged
from the first round, apart from small changes in the commit log
wording. Patch 3 now includes parentheses in the macro
definition. Patches 4 and 5 are dropped.

4/4 is mostly a minor optimization, omitting a tiny bit of dead code
from the compiler binary. But the preprocessor directives also serve
as a reminder that the custom vxworks functions will not be used if
the compiler is built with --enable-initfini-array.

Rasmus Villemoes (4):
  vxworks: add target/h/wrn/coreip to the set of system include paths
  libgcc: add crt{begin,end} for powerpc-wrs-vxworks target
  vxworks: enable use of .init_array/.fini_array for cdtors
  vxworks: don't define vxworks_asm_out_constructor when using
.init_array

 gcc/config/vxworks.c |  9 +++--
 gcc/config/vxworks.h | 21 +++--
 libgcc/config.host   |  1 +
 3 files changed, 23 insertions(+), 8 deletions(-)

-- 
2.16.4