[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

rename issue

Ralf Corsepius wrote:
> Joel Sherrill wrote:
>   
>> Looks like we are heading for a new spin of the 4.10 tools
>> soon.
>>   
>>     
> Yep, .... I am going to address these issues sequentially.
>   
>>   + drop - DMISSING_SYSCALL_NAMES from configure.host
>>   
>>     
> Having cross-checked your proposal, I leaned to agree with your proposal 
> and am about to launch a toolchain spin.
>
> Please test this toolchain! Though this patch is a one-liner, this step 
> is quite intrusive, and is not unlikely to have (so far) unconsidered 
> side-effects.
>
>   
Yeah!  This one worried me.  It could easily turn up a LOT of
stuff.
>>   + ALIGN issue not being 8 on SPARC
>>   
>>     
> Patch? I believe to recall having read an email on this issue, but am 
> not sure about this nor can I find it anymore.
>
>   
I'm not sure about the right solution.  I am leaning on changing
the definition of  ALIGN in param.h from:

#define ALIGNBYTES    (sizeof(int) - 1)

to

#define ALIGNBYTES    (MALLOC_ALIGNMENT - 1)

MALLOC_ALIGNMENT appears to be conditionally defined
in sys/config.h but not for the sparc.  :(

I think that's the right solution but would like feedback before
implementing it.

>>   + inttypes.h warning
>>   
>>     
> WIP on my part, but no patch available so far.
>   
OK.  Chris and I were on a warning hunt and these showed up
in cpukit.  They will disappear again when you get the patch.

--joel