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

Christian Thalinger christian.thalinger at oracle.com
Mon Dec 28 20:09:55 UTC 2015


Thanks, Vladimir.

> On Dec 28, 2015, at 10:07 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> 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() {
>> 



More information about the hotspot-compiler-dev mailing list