RFR (XXS): 8146245: compiler/jvmci/ tests fail: java.lang.AssertionError: minimum config for aarch64

[Vladimir Kozlov]

Looks good.

Thanks,
Vladimir

On 12/28/15 10:37 AM, Christian Thalinger wrote:
> https://bugs.openjdk.java.net/browse/JDK-8146245
>
> Currently no CPU features are computed.  I think we should just remove
> the assert until we actually do that.
>
> diff -r 4a0c467b12bb
> src/jdk.vm.ci/share/classes/jdk.vm.ci.aarch64/src/jdk/vm/ci/aarch64/AArch64.java
> ---
> a/src/jdk.vm.ci/share/classes/jdk.vm.ci.aarch64/src/jdk/vm/ci/aarch64/AArch64.javaSat
> Dec 26 16:59:26 2015 +0100
> +++
> b/src/jdk.vm.ci/share/classes/jdk.vm.ci.aarch64/src/jdk/vm/ci/aarch64/AArch64.javaMon
> Dec 28 08:33:04 2015 -1000
> @@ -175,11 +175,9 @@ public class AArch64 extends Architectur
>       private final EnumSet<Flag> flags;
>
>
>       public AArch64(EnumSet<CPUFeature> features, EnumSet<Flag> flags) {
> -        super("aarch64", AArch64Kind.QWORD, ByteOrder.LITTLE_ENDIAN, /*
> unalignedMemoryAccess */true, allRegisters, /* implicitMemoryBarriers
> */0, /* nativeCallDisplacementOffset */0,
> -                        /* returnAddressSize */0);
> +        super("aarch64", AArch64Kind.QWORD, ByteOrder.LITTLE_ENDIAN,
> true, allRegisters, 0, 0, 0);
>           this.features = features;
>           this.flags = flags;
> -        assert features.contains(CPUFeature.FP) : "minimum config for
> aarch64";
>       }
>
>
>       public EnumSet<CPUFeature> getFeatures() {
>