[oe] [PATCH 1/2] mozjs: set -fno-schedule-insns2 on both CFLAGS and CXXFLAGS

[Khem Raj]

Hi Ricardo


On Tue, Jul 31, 2018 at 8:06 PM Ricardo Salveti <ricardo at foundries.io> wrote:
>
> Mozjs fails to start on several architectures due broken build
> optimizations when building with GCC >= 6.
>
> Set -fno-schedule-insns2 as workaround (flag also used by Firefox and
> Debian).
>
> Signed-off-by: Ricardo Salveti <ricardo at foundries.io>
> ---
>  meta-oe/recipes-extended/mozjs/mozjs_52.8.1.bb | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/meta-oe/recipes-extended/mozjs/mozjs_52.8.1.bb b/meta-oe/recipes-extended/mozjs/mozjs_52.8.1.bb
> index da80b63..adc27ac 100644
> --- a/meta-oe/recipes-extended/mozjs/mozjs_52.8.1.bb
> +++ b/meta-oe/recipes-extended/mozjs/mozjs_52.8.1.bb
> @@ -30,6 +30,11 @@ CONFLICT_DISTRO_FEATURES_mipsarchn32 = "ld-is-gold"
>
>  DEPENDS += "nspr zlib"
>
> +# Avoid broken JIT on several architectures (from Firefox and Debian)
> +# https://bugzilla.mozilla.org/show_bug.cgi?id=1245783
> +CFLAGS += "-fno-schedule-insns2"
> +CXXFLAGS += "-fno-schedule-insns2"
> +

Could you test with -fno-tree-vrp -fno-strict-aliasing
-fno-delete-null-pointer-checks instead of -fno-schedule-insns2

>  # nspr's package-config is ignored so set libs manually
>  EXTRA_OECONF = " \
>      --target=${TARGET_SYS} \
> --
> 2.7.4
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel