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
> 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
Description: S/MIME cryptographic signature