On Thu, 9 Apr 2026 07:08:21 GMT, Jatin Bhateja <[email protected]> wrote:
> ROL, ROR, LSHL, ASHR, LSHR in VectorOperators.java were declared with > VO_SHIFT but missing VO_NOFP, allowing them to pass the opCode() guard on > floating-point vectors (DoubleVector, FloatVector). > > This bug fix patch adds VO_NOFP to all five declarations. > > As a result, C2 only compiles the hot code path comprising of opCode() check, > sees the illegalOperation throw an exception, cold path comprising of > VectorSupport.binaryOp intrinsic not compiled and thus flow never reaches > vop2ideal() which generates a fatal error if shift class operation is > performed over floating point type > > Please review and share your feedback. > > Best Regards, > Jatin > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: d5d8532c Author: Jatin Bhateja <[email protected]> URL: https://git.openjdk.org/jdk/commit/d5d8532ca2ca5cbdcb80af43fb1a192c20af9914 Stats: 163 lines in 3 files changed: 121 ins; 0 del; 42 mod 8381579: C2: "fatal error: LROTATE: double" when using VectorAPI Reviewed-by: mhaessig, qamai ------------- PR: https://git.openjdk.org/jdk/pull/30642
