Re: [PATCH v2 1/1] OvmfPkg/build.sh: select the GCC49 toolchain settings for gcc-7.*

[Jordan Justen <jordan.l.justen@intel.com>]

Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

On 2017-08-15 10:09:14, Laszlo Ersek wrote:
> When UefiCpuPkg/MpInitLib is built for X64 with gcc-7, using the DEBUG
> build target and the GCC5 toolchain settings, a C-language assignment is
> miscompiled such that the initial AP startup hangs in CpuMpPei (X64) or
> CpuDxe (Ia32X64). See <https://bugzilla.tianocore.org/show_bug.cgi?id=671>
> for a detailed analysis of the symptoms, and for mailing list links.
> 
> This issue has been reported several times (one example is
> <https://bugzilla.tianocore.org/show_bug.cgi?id=657>). Until we (or the
> upstream gcc developers) figure out how to dissuade gcc-7 from the
> miscompilation, pick the GCC49 toolchain in "build.sh" for gcc-7.*.
> 
> Cc: Alex Williamson <alex.williamson@redhat.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
> ---
>  OvmfPkg/build.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/OvmfPkg/build.sh b/OvmfPkg/build.sh
> index 682155202564..5868d3b8c498 100755
> --- a/OvmfPkg/build.sh
> +++ b/OvmfPkg/build.sh
> @@ -102,7 +102,7 @@ case `uname` in
>        4.8.*)
>          TARGET_TOOLS=GCC48
>          ;;
> -      4.9.*|6.[0-2].*)
> +      4.9.*|6.[0-2].*|7.*)
>          TARGET_TOOLS=GCC49
>          ;;
>        *)
> -- 
> 2.14.1.3.gb7cf6e02401b
>