8275643: C2's unaryOp vector intrinsic does not properly handle LongV…

…ector.neg

Reviewed-by: chagedorn, sviswanathan

src/hotspot/share/prims/vectorSupport.cpp

@@ -292,6 +292,7 @@ int VectorSupport::vop2ideal(jint id, BasicType bt) {
         case T_BYTE:   // fall-through
         case T_SHORT:  // fall-through
         case T_INT:    return Op_NegI;
+        case T_LONG:   return Op_NegL;
         case T_FLOAT:  return Op_NegF;
         case T_DOUBLE: return Op_NegD;
         default: fatal("NEG: %s", type2name(bt));

test/hotspot/jtreg/compiler/vectorapi/TestLongVectorNeg.java

@@ -0,0 +1,50 @@
+/*
+ * Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+package compiler.vectorapi;
+
+import jdk.incubator.vector.LongVector;
+
+/*
+ * @test
+ * @bug 8275643
+ * @summary Test that LongVector.neg is properly handled by the _VectorUnaryOp C2 intrinsic
+ * @modules jdk.incubator.vector
+ * @run main/othervm -XX:-TieredCompilation -XX:+AlwaysIncrementalInline -Xbatch
+ *                   -XX:CompileCommand=dontinline,compiler.vectorapi.TestLongVectorNeg::test
+ *                   compiler.vectorapi.TestLongVectorNeg
+ */
+public class TestLongVectorNeg {
+
+    static LongVector test(LongVector v) {
+        return v.neg();
+    }
+
+    public static void main(String[] args) {
+        LongVector v = LongVector.zero(LongVector.SPECIES_128);
+        for (int i = 0; i < 50_000; ++i) {
+            v.abs(); // Warmup to make sure the !VO_SPECIAL code path is taken as well
+            test(v);
+        }
+    }
+}