Messages in this thread |  | | | Date | Thu, 12 Apr 2012 21:08:36 +0200 | | Subject | Re: [PATCH 2/2] Add CONFIG_READABLE_ASM | | From | Geert Uytterhoeven <> |
| |
Hi Andi,
On Wed, Mar 28, 2012 at 20:51, Andi Kleen <andi@firstfloor.org> wrote: > +ifdef CONFIG_READABLE_ASM > +# Disable optimizations that make assembler listings hard to read. > +# reorder blocks reorders the control in the function > +# ipa clone creates specialized cloned functions > +# partial inlining inlines only parts of functions > +KBUILD_CFLAGS += $(call cc-option,-fno-reorder-blocks,) \ > + $(call cc-option,-fno-ipa-cp-clone,) \ > + $(call cc-option,-fno-partial-inlining) > +endif
This (now in linux-next) causes m68k/allmodconfig to fail for me:
CHK include/linux/version.h CHK include/generated/utsrelease.h CC kernel/bounds.s cc1: error: unrecognized command line option "-fno-ipa-cp-clone"
Somehow, "$(call cc-option,-fno-ipa-cp-clone,)" doesn't detect that my toolchain (gcc version 4.1.2 20061115 (prerelease) (Ubuntu 4.1.1-21)) doesn't support this option.
I tried playing with the trailing comma (why do the first 2 tests have it, and the 3rd one doesn't?), but that didn't make a difference.
linux-next builds (using a newer toolchain) don't show this.
Gr{oetje,eeting}s, Geert
-- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |