Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1206062

Re: [PATCH] Make alignment cflags configurable.

From David Woodhouse <dwmw2@infradead.org>
Newsgroups linux.kernel
Subject Re: [PATCH] Make alignment cflags configurable.
Date 2015-08-12 15:00 +0200
Message-ID <pWDNo-4Q9-13@gated-at.bofh.it> (permalink)
References <pWDu1-4ta-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] Make alignment cflags configurable. dl9pf@gmx.de - 2015-08-12 14:40 +0200
  Re: [PATCH] Make alignment cflags configurable. David Woodhouse <dwmw2@infradead.org> - 2015-08-12 15:00 +0200
    Re: [PATCH] Make alignment cflags configurable. Jan-Simon Moeller <dl9pf@gmx.de> - 2015-08-13 00:40 +0200
      Re: [PATCH] Make alignment cflags configurable. "H. Peter Anvin" <hpa@zytor.com> - 2015-08-13 00:40 +0200
        Re: [llvmlinux] [PATCH] Make alignment cflags configurable. Jan-Simon Moeller <dl9pf@gmx.de> - 2015-08-13 01:30 +0200
          Re: [llvmlinux] [PATCH] Make alignment cflags configurable. David Woodhouse <dwmw2@infradead.org> - 2015-08-13 01:40 +0200
            Re: [llvmlinux] [PATCH] Make alignment cflags configurable. Jan-Simon Moeller <dl9pf@gmx.de> - 2015-08-13 02:10 +0200
              Re: [llvmlinux] [PATCH] Make alignment cflags configurable. David Woodhouse <dwmw2@infradead.org> - 2015-08-13 02:10 +0200

csiph-web