On 17 December 2015 at 11:01, Ard Biesheuvel <[email protected]> wrote:
> Since we are combining Clang with the GNU linker, make that explicit in
> the target triplet. This affects certain builtin defines and other compiler
> behavior that may be unspecified otherwise.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <[email protected]>
> ---
>  BaseTools/Conf/tools_def.template | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/BaseTools/Conf/tools_def.template 
> b/BaseTools/Conf/tools_def.template
> index ea321476a496..0cc85a6f359d 100644
> --- a/BaseTools/Conf/tools_def.template
> +++ b/BaseTools/Conf/tools_def.template
> @@ -5181,8 +5181,8 @@ RELEASE_GCC49_AARCH64_CC_FLAGS   = 
> DEF(GCC49_AARCH64_CC_FLAGS) -Wno-unused-but-s
>  *_CLANG35_*_ASLCC_PATH           = ENV(CLANG35_BIN)clang
>  *_CLANG35_*_ASLPP_PATH           = ENV(CLANG35_BIN)clang
>
> -DEFINE CLANG35_ARM_TARGET        = -target armv7-a
> -DEFINE CLANG35_AARCH64_TARGET    = -target aarch64
> +DEFINE CLANG35_ARM_TARGET        = -target armv7-none-linux-gnueabi
> +DEFINE CLANG35_AARCH64_TARGET    = -target aarch64-none-linux-gnu
>
>  DEFINE CLANG35_WARNING_OVERRIDES = -Wno-parentheses-equality 
> -Wno-tautological-compare -Wno-tautological-constant-out-of-range-compare 
> -Wno-empty-body
>  DEFINE CLANG35_ARM_CC_FLAGS      = DEF(GCC_ARM_CC_FLAGS) 
> DEF(CLANG35_ARM_TARGET) -mstrict-align -mllvm -arm-use-movt=0 
> DEF(CLANG35_WARNING_OVERRIDES)

@Liming, Yonghong: are you ok with this patch? This needs to be
applied before the patch that sets the target for the various PP flags
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to