Re: [PATCH] Make alignment cflags configurable.

[David Woodhouse]

On Wed, 2015-08-12 at 14:32 +0200, dl9pf@xxxxxx wrote:
> From: Jan-Simon MÃller <dl9pf@xxxxxx>
> 
> This patch adds switches for
> -falign-jumps=1
> and
> -falign-loops=1
> 
> Default is off by intention to allow seamless operation.
> 
> Signed-off-by: Jan-Simon MÃller <dl9pf@xxxxxx>

You could mention that this is to fix the clang build. But why is it
needed? It isn't that clang just doesn't accept the option, is it?
Otherwise we could just use $(call cc-option, -falign-jumps=1) etc.

Did you get to the bottom of the clang failure here? Just turning this
off without a coherent explanation doesn't seem like the right thing to
do.

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@xxxxxxxxx                              Intel Corporation

Attachment: smime.p7s
Description: S/MIME cryptographic signature