Thu, 19 Mar 2015 19:53:34 +0100
8074869: C2 code generator can replace -0.0f with +0.0f on Linux
Summary: Instead of 'fpclass', use cast float->int and double->long to check if value is +0.0f and +0.0d, respectively.
Reviewed-by: kvn, simonis, dlong
zmajo@7638 | 1 | /* |
zmajo@7638 | 2 | * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved. |
zmajo@7638 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
zmajo@7638 | 4 | * |
zmajo@7638 | 5 | * This code is free software; you can redistribute it and/or modify it |
zmajo@7638 | 6 | * under the terms of the GNU General Public License version 2 only, as |
zmajo@7638 | 7 | * published by the Free Software Foundation. |
zmajo@7638 | 8 | * |
zmajo@7638 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
zmajo@7638 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
zmajo@7638 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
zmajo@7638 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
zmajo@7638 | 13 | * accompanied this code). |
zmajo@7638 | 14 | * |
zmajo@7638 | 15 | * You should have received a copy of the GNU General Public License version |
zmajo@7638 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
zmajo@7638 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
zmajo@7638 | 18 | * |
zmajo@7638 | 19 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
zmajo@7638 | 20 | * or visit www.oracle.com if you need additional information or have any |
zmajo@7638 | 21 | * questions. |
zmajo@7638 | 22 | * |
zmajo@7638 | 23 | */ |
zmajo@7638 | 24 | |
zmajo@7638 | 25 | /** |
zmajo@7638 | 26 | * @test |
zmajo@7638 | 27 | * @bug 8074869 |
zmajo@7638 | 28 | * @summary C2 code generator can replace -0.0f with +0.0f on Linux |
zmajo@7638 | 29 | * @run main ConstFPVectorization 8 |
zmajo@7638 | 30 | * @author volker.simonis@gmail.com |
zmajo@7638 | 31 | * |
zmajo@7638 | 32 | */ |
zmajo@7638 | 33 | |
zmajo@7638 | 34 | public class ConstFPVectorization { |
zmajo@7638 | 35 | |
zmajo@7638 | 36 | static float[] f = new float[16]; |
zmajo@7638 | 37 | static double[] d = new double[16]; |
zmajo@7638 | 38 | |
zmajo@7638 | 39 | static void floatLoop(int count) { |
zmajo@7638 | 40 | for (int i = 0; i < count; i++) { |
zmajo@7638 | 41 | f[i] = -0.0f; |
zmajo@7638 | 42 | } |
zmajo@7638 | 43 | } |
zmajo@7638 | 44 | |
zmajo@7638 | 45 | static void doubleLoop(int count) { |
zmajo@7638 | 46 | for (int i = 0; i < count; i++) { |
zmajo@7638 | 47 | d[i] = -0.0d; |
zmajo@7638 | 48 | } |
zmajo@7638 | 49 | } |
zmajo@7638 | 50 | |
zmajo@7638 | 51 | public static void main(String args[]) { |
zmajo@7638 | 52 | for (int i = 0; i < 10_000; i++) { |
zmajo@7638 | 53 | floatLoop(Integer.parseInt(args[0])); |
zmajo@7638 | 54 | doubleLoop(Integer.parseInt(args[0])); |
zmajo@7638 | 55 | } |
zmajo@7638 | 56 | for (int i = 0; i < Integer.parseInt(args[0]); i++) { |
zmajo@7638 | 57 | if (Float.floatToRawIntBits(f[i]) != Float.floatToRawIntBits(-0.0f)) |
zmajo@7638 | 58 | throw new Error("Float error at index " + i); |
zmajo@7638 | 59 | if (Double.doubleToRawLongBits(d[i]) != Double.doubleToRawLongBits(-0.0d)) |
zmajo@7638 | 60 | throw new Error("Double error at index " + i); |
zmajo@7638 | 61 | } |
zmajo@7638 | 62 | } |
zmajo@7638 | 63 | } |