[coreboot-gerrit] Patch set updated for coreboot: libpayload: Use top level xcompile
Stefan Reinauer (stefan.reinauer@coreboot.org)
gerrit at coreboot.org
Thu Jul 9 02:06:31 CEST 2015
Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10863
-gerrit
commit cbfd1f67c503e643b56b4734e7120f2faa278b52
Author: Stefan Reinauer <stefan.reinauer at coreboot.org>
Date: Thu Jul 9 00:26:49 2015 +0200
libpayload: Use top level xcompile
Instead of having a second copy that already within 2-3 days
becamer quite outdated, use the same xcompile copy for coreboot
and libpayload, as we do with Kconfig already.
This requires a simple change to the top level xcompile to understand
both CONFIG_COMPILER_GCC and CONFIG_LP_COMPILER_GCC (only one of
them will occur at the same time)
libpayload's .xcompile target was moved later so that it can make use
of $(top)
Change-Id: I44001067f551cd0776dd303cbaeaa40eb3d5c1db
Signed-off-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
---
payloads/libpayload/Makefile | 28 +--
payloads/libpayload/util/xcompile/xcompile | 337 -----------------------------
util/xcompile/xcompile | 2 +-
3 files changed, 15 insertions(+), 352 deletions(-)
diff --git a/payloads/libpayload/Makefile b/payloads/libpayload/Makefile
index 2b9bbcb..f3544d4 100644
--- a/payloads/libpayload/Makefile
+++ b/payloads/libpayload/Makefile
@@ -34,20 +34,6 @@ ifeq ($(INNER_SCANBUILD),y)
CC_real:=$(CC)
endif
-# in addition to the dependency below, create the file if it doesn't exist
-# to silence stupid warnings about a file that would be generated anyway.
-$(if $(wildcard .xcompile),,$(eval $(shell util/xcompile/xcompile $(XGCCPATH) > .xcompile)))
-
-.xcompile: util/xcompile/xcompile
- $< $(XGCCPATH) > $@.tmp
- \mv -f $@.tmp $@ 2> /dev/null
-
-ifeq ($(INNER_SCANBUILD),y)
-CC:=$(CC_real)
-HOSTCC:=$(CC_real) --hostcc
-HOSTCXX:=$(CC_real) --hostcxx
-endif
-
export top := $(CURDIR)
export src := src
export srck := $(abspath $(top)/../../util/kconfig)
@@ -95,6 +81,20 @@ DOXYGEN_OUTPUT_DIR := doxygen
all: real-all
+# in addition to the dependency below, create the file if it doesn't exist
+# to silence stupid warnings about a file that would be generated anyway.
+$(if $(wildcard .xcompile),,$(eval $(shell $(top)/../../util/xcompile/xcompile $(XGCCPATH) > .xcompile)))
+
+.xcompile: $(top)/../../util/xcompile/xcompile
+ $< $(XGCCPATH) > $@.tmp
+ \mv -f $@.tmp $@ 2> /dev/null
+
+ifeq ($(INNER_SCANBUILD),y)
+CC:=$(CC_real)
+HOSTCC:=$(CC_real) --hostcc
+HOSTCXX:=$(CC_real) --hostcxx
+endif
+
# This include must come _before_ the pattern rules below!
# Order _does_ matter for pattern rules.
include $(srck)/Makefile
diff --git a/payloads/libpayload/util/xcompile/xcompile b/payloads/libpayload/util/xcompile/xcompile
deleted file mode 100755
index b6759a9..0000000
--- a/payloads/libpayload/util/xcompile/xcompile
+++ /dev/null
@@ -1,337 +0,0 @@
-#!/bin/bash
-#
-# This file is part of the coreboot project.
-#
-# Copyright (C) 2007-2010 coresystems GmbH
-# Copyright (C) 2012 Google Inc
-#
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; version 2 of the License.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc.
-#
-
-TMPFILE=""
-XGCCPATH=${1:-"`pwd`/util/crossgcc/xgcc/bin/"}
-
-die() {
- echo "ERROR: $*" >&2
- exit 1
-}
-
-clean_up() {
- if [ -n "$TMPFILE" ]; then
- rm -f "$TMPFILE" "$TMPFILE.c" "$TMPFILE.o"
- fi
-}
-
-# Create temporary file(s).
-TMPFILE="$(mktemp /tmp/temp.XXXX 2>/dev/null || echo /tmp/temp.78gOIUGz)"
-touch "$TMPFILE"
-trap clean_up EXIT
-
-
-program_exists() {
- type "$1" >/dev/null 2>&1
-}
-
-
-if [ "$(${XGCCPATH}/iasl 2>/dev/null | grep -c ACPI)" -gt 0 ]; then
- IASL=${XGCCPATH}iasl
-elif [ "$(iasl 2>/dev/null | grep -c ACPI)" -gt 0 ]; then
- IASL=iasl
-else
- die "no iasl found"
-fi
-
-if program_exists gcc; then
- HOSTCC=gcc
-elif program_exists cc; then
- HOSTCC=cc
-else
- die "no host compiler found"
-fi
-
-cat <<EOF
-# platform agnostic and host tools
-IASL:=${IASL}
-HOSTCC?=${HOSTCC}
-
-EOF
-
-testcc() {
- local tmp_c="$TMPFILE.c"
- local tmp_o="$TMPFILE.o"
- rm -f "$tmp_c" "$tmp_o"
- echo "void _start(void) {}" >"$tmp_c"
- "$1" -nostdlib -Werror $2 "$tmp_c" -o "$tmp_o" >/dev/null 2>&1
-}
-
-testas() {
- local gccprefix="$1"
- local twidth="$2"
- local arch="$3"
- local use_dash_twidth="$4"
- local endian="$5"
- local obj_file="$TMPFILE.o"
- local full_arch="elf$twidth-$arch"
-
- rm -f "$obj_file"
- [ -n "$use_dash_twidth" ] && use_dash_twidth="--$twidth"
- [ -n "$endian" ] && endian="-$endian"
- ${gccprefix}as $use_dash_twidth $endian -o "$obj_file" $TMPFILE \
- 2>/dev/null || return 1
-
- # Check output content type.
- local obj_type="$(${gccprefix}objdump -p $obj_file)"
- local obj_arch="$(expr "$obj_type" : '.*format \(.[a-z0-9-]*\)')"
- [ "$obj_arch" = "$full_arch" ] || return 1
-
- # Architecture matched.
- GCCPREFIX="$gccprefix"
-
- unset ASFLAGS LDFLAGS
- unset CFLAGS_GCC CFLAGS_CLANG
-
- if [ -n "$use_dash_twidth" ]; then
- ASFLAGS="--$twidth"
- CFLAGS_GCC="-m$twidth"
- CFLAGS_CLANG="-m$twidth"
- LDFLAGS="-b $full_arch"
-
- fi
-
- # Special parameters only available in dash_twidth mode.
- [ -n "$use_dash_twidth" ] && case "$full_arch" in
- "elf32-i386" )
- LDFLAGS="$LDFLAGS -melf_i386"
- CFLAGS_GCC="$CFLAGS_GCC -Wl,-b,elf32-i386 -Wl,-melf_i386"
- CFLAGS_CLANG="$CFLAGS_GCC -Wl,-b,elf32-i386 -Wl,-melf_i386"
- ;;
- esac
-
- return 0
-}
-
-detect_special_flags() {
- local architecture="$1"
- # GCC 4.6 is much more picky about unused variables.
- # Turn off it's warnings for now:
- testcc "$GCC" "$CFLAGS_GCC -Wno-unused-but-set-variable " &&
- CFLAGS_GCC="$CFLAGS_GCC -Wno-unused-but-set-variable "
-
- # Check for an operational -m32/-m64
- testcc "$GCC" "$CFLAGS_GCC -m$TWIDTH " &&
- CFLAGS_GCC="$CFLAGS_GCC -m$TWIDTH "
-
- # Use bfd linker instead of gold if available:
- testcc "$GCC" "$CFLAGS_GCC -fuse-ld=bfd" &&
- CFLAGS_GCC="$CFLAGS_GCC -fuse-ld=bfd" && LINKER_SUFFIX='.bfd'
-
- testcc "$GCC" "$CFLAGS_GCC -fno-stack-protector"&&
- CFLAGS_GCC="$CFLAGS_GCC -fno-stack-protector"
- testcc "$GCC" "$CFLAGS_GCC -Wl,--build-id=none" &&
- CFLAGS_GCC="$CFLAGS_GCC -Wl,--build-id=none"
-
- case "$architecture" in
- x86)
- testcc "$GCC" "$CFLAGS_GCC -Wa,--divide" &&
- CFLAGS_GCC="$CFLAGS_GCC -Wa,--divide"
- testcc "$CLANG" "$CFLAGS_CLANG -Wa,--divide" &&
- CFLAGS_CLANG="$CFLAGS_CLANG -Wa,--divide"
- # Always build for i686 -- no sse/mmx instructions since SMM
- # modules are compiled using these flags. Note that this
- # doesn't prevent a project using xcompile to explicitly
- # specify -mmsse, etc flags.
- CFLAGS_GCC="$CFLAGS_GCC -march=i686"
- ;;
- x64)
- testcc "$GCC" "$CFLAGS_GCC -Wa,--divide" &&
- CFLAGS_GCC="$CFLAGS_GCC -Wa,--divide"
- testcc "$CLANG" "$CFLAGS_CLANG -Wa,--divide" &&
- CFLAGS_CLANG="$CFLAGS_CLANG -Wa,--divide"
- ;;
- mipsel)
- testcc "$GCC" "$CFLAGS_GCC -mno-abicalls -fno-pic" && \
- CFLAGS_GCC+=" -mno-abicalls -fno-pic"
-
- # Enforce little endian mode.
- testcc "$GCC" "$CFLAGS_GCC -EL" && \
- CFLAGS_GCC+=" -EL"
- ;;
- esac
-}
-
-detect_compiler_runtime() {
- test -z "$CLANG" || \
- CC_RT_CLANG="`${CLANG} ${CFLAGS} -print-librt-file-name 2>/dev/null`"
- test -z "$GCC" || \
- CC_RT_GCC="`${GCC} ${CFLAGS} -print-libgcc-file-name`"
-}
-
-report_arch_toolchain() {
- cat <<EOF
-# elf${TWIDTH}-${TBFDARCH} toolchain (${GCC})
-ARCH_SUPPORTED+=${TARCH}
-SUBARCH_SUPPORTED+=${TSUPP-${TARCH}}
-ifeq (\$(CONFIG_LP_COMPILER_GCC),y)
-CC_${TARCH}:=${GCC}
-CFLAGS_${TARCH}:=${CFLAGS_GCC}
-COMPILER_RT_${TARCH}:=${CC_RT_GCC}
-COMPILER_RT_FLAGS_${TARCH}:=${CC_RT_EXTRA_GCC}
-else
-CC_${TARCH}:=${CLANG}
-CFLAGS_${TARCH}:=${CFLAGS_CLANG}
-CFLAGS_${TARCH}+=-no-integrated-as -Qunused-arguments -m${TWIDTH}
-# tone down compiler warnings
-CFLAGS_${TARCH}+=-Wno-unused-variable -Wno-unused-function -Wno-tautological-compare -Wno-shift-overflow
-COMPILER_RT_${TARCH}:=${CC_RT_CLANG}
-COMPILER_RT_FLAGS_${TARCH}:=${CC_RT_EXTRA_CLANG}
-endif
-CPP_${TARCH}:=${GCCPREFIX}cpp
-AS_${TARCH}:=${GCCPREFIX}as ${ASFLAGS}
-LD_${TARCH}:=${GCCPREFIX}ld${LINKER_SUFFIX} ${LDFLAGS}
-NM_${TARCH}:=${GCCPREFIX}nm
-OBJCOPY_${TARCH}:=${GCCPREFIX}objcopy
-OBJDUMP_${TARCH}:=${GCCPREFIX}objdump
-READELF_${TARCH}:=${GCCPREFIX}readelf
-STRIP_${TARCH}:=${GCCPREFIX}strip
-AR_${TARCH}:=${GCCPREFIX}ar
-CROSS_COMPILE_${TARCH}:=${GCCPREFIX}
-
-EOF
-}
-
-# Architecture definitions
-SUPPORTED_ARCHITECTURES="arm arm64 mipsel riscv x64 x86"
-
-arch_config_arm() {
- TARCH="arm"
- TBFDARCHS="littlearm"
- TCLIST="armv7a armv7-a"
- TWIDTH="32"
- TSUPP="arm armv4 armv7 armv7_m"
- TABI="eabi"
-}
-
-arch_config_arm64() {
- TARCH="arm64"
- TBFDARCHS="littleaarch64"
- TCLIST="aarch64"
- TWIDTH="64"
- TSUPP="arm64 armv8_64"
- TABI="elf"
-}
-
-arch_config_riscv() {
- TARCH="riscv"
- TBFDARCHS="littleriscv"
- TCLIST="riscv"
- TWIDTH="64"
- TABI="elf"
-}
-
-arch_config_x64() {
- TARCH="x86_64"
- TBFDARCHS="x86-64"
- TCLIST="x86_64"
- TWIDTH="64"
- TABI="elf"
-}
-
-arch_config_x86() {
- TARCH="x86_32"
- TBFDARCHS="i386"
- TCLIST="i386 x86_64"
- TWIDTH="32"
- TABI="elf"
- CC_RT_EXTRA_GCC="--wrap __divdi3 --wrap __udivdi3 --wrap __moddi3 --wrap __umoddi3"
-}
-
-arch_config_mipsel() {
- TARCH="mips"
- TBFDARCHS="tradlittlemips littlemips"
- TCLIST="mipsel"
- TWIDTH="32"
- TSUPP="mips mipsel"
- TABI="elf"
- TENDIAN="EL"
-}
-
-test_architecture() {
- local architecture=$1
- local endian gccprefix search
-
- GCCPREFIX="invalid"
- unset TABI TARCH TBFDARCH TCLIST TENDIAN TSUPP TWIDTH
- unset CC_RT_EXTRA_GCC CC_RT_EXTRA_CLANG
- unset GCC CLANG
- if type arch_config_$architecture > /dev/null; then
- arch_config_$architecture
- else
- die "no architecture definition for $architecture"
- fi
-
- # To override toolchain, define CROSS_COMPILE_$arch or CROSS_COMPILE as
- # environment variable.
- # Ex: CROSS_COMPILE_arm="armv7a-cros-linux-gnueabi-"
- # CROSS_COMPILE_x86="i686-pc-linux-gnu-"
- search="$(eval echo \$CROSS_COMPILE_$architecture 2>/dev/null)"
- search="$search $CROSS_COMPILE"
- for toolchain in $TCLIST; do
- search="$search $XGCCPATH$toolchain-$TABI-"
- search="$search $toolchain-$TABI-"
- search="$search $toolchain-linux-gnu-"
- search="$search $toolchain-"
- done
- echo "# $architecture TARCH_SEARCH=$search"
-
- # Search toolchain by checking assembler capability.
- for TBFDARCH in $TBFDARCHS; do
- for gccprefix in $search ""; do
- program_exists "${gccprefix}as" || continue
- for endian in $TENDIAN ""; do
- testas "$gccprefix" "$TWIDTH" "$TBFDARCH" \
- "" "$endian" && break 3
- testas "$gccprefix" "$TWIDTH" "$TBFDARCH" \
- "TRUE" "$endian" && break 3
- done
- done
- done
- if [ "invalid" != "$GCCPREFIX" ]; then
- GCC="${GCCPREFIX}gcc"
- fi
-
- for clang_arch in $TCLIST invalid; do
- testcc "clang" "-target ${clang_arch}-$TABI -c" && break
- done
-
- if [ "invalid" != "$clang_arch" ]; then
- # FIXME: this may break in a clang && !gcc configuration,
- # but that's more of a clang limitation. Let's be optimistic
- # that this will change in the future.
- CLANG="clang -target ${clang_arch}-${TABI} -ccc-gcc-name ${GCC}"
- fi
-
- if [ -z "$GCC" -a -z "$CLANG" ]; then
- echo "Warning: no suitable compiler for $architecture." >&2
- return 1
- fi
-}
-
-# This loops over all supported architectures.
-for architecture in $SUPPORTED_ARCHITECTURES; do
- if test_architecture $architecture; then
- detect_special_flags "$architecture"
- detect_compiler_runtime "$architecture"
- report_arch_toolchain
- fi
-done
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index b11f7dd..c1c42af 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -174,7 +174,7 @@ report_arch_toolchain() {
# elf${TWIDTH}-${TBFDARCH} toolchain (${GCC})
ARCH_SUPPORTED+=${TARCH}
SUBARCH_SUPPORTED+=${TSUPP-${TARCH}}
-ifeq (\$(CONFIG_COMPILER_GCC),y)
+ifeq (\$(CONFIG_COMPILER_GCC)\$(CONFIG_LP_COMPILER_GCC),y)
CC_${TARCH}:=${GCC}
CFLAGS_${TARCH}:=${CFLAGS_GCC}
COMPILER_RT_${TARCH}:=${CC_RT_GCC}
More information about the coreboot-gerrit
mailing list