Re: [PATCH] Make alignment cflags configurable.

From: David Woodhouse
Date: Wed Aug 12 2015 - 08:50:57 EST

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

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

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