[8u] Request for approval for JDK-8169201: Montgomery multiply intrinsic should use correct name

Aleksey Shipilev shade at redhat.com
Wed Mar 21 15:54:33 UTC 2018


It is a really trivial patch:
  http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/110b1774a359

diff -r 96e9e479ff8a src/share/vm/opto/library_call.cpp
--- a/src/share/vm/opto/library_call.cpp	Thu Aug 24 01:27:02 2017 -0400
+++ b/src/share/vm/opto/library_call.cpp	Wed Mar 21 16:49:39 2018 +0100
@@ -6026,7 +6026,7 @@
   }

   assert(UseMontgomeryMultiplyIntrinsic, "not implemented on this platform");
-  const char* stubName = "montgomery_square";
+  const char* stubName = "montgomery_multiply";

   assert(callee()->signature()->size() == 7, "montgomeryMultiply has 7 parameters");


It applies with a bit of fuzz to jdk8-dev.

JDK 9 bug:
  https://bugs.openjdk.java.net/browse/JDK-8169201

JDK 9 RFR:
  http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-November/024899.html

We need this to avoid weird bugs in code that relies on intrinsic names:
  http://mail.openjdk.java.net/pipermail/aarch64-port-dev/2018-March/005424.html

Thanks,
-Aleksey



More information about the jdk8u-dev mailing list