[llvmlinux] [PATCH] Make alignment cflags configurable.

David Woodhouse dwmw2 at infradead.org
Wed Aug 12 12:50:44 UTC 2015


On Wed, 2015-08-12 at 14:32 +0200, dl9pf at gmx.de wrote:
> From: Jan-Simon Möller <dl9pf at gmx.de>
> 
> 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 at gmx.de>

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 at intel.com                              Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 5691 bytes
Desc: not available
URL: <http://lists.linuxfoundation.org/pipermail/llvmlinux/attachments/20150812/1ad43ae8/attachment.bin>


More information about the LLVMLinux mailing list