Update pom.xml to reflect JDK 25 Vector API requirements and add necessary compiler arguments

Author: mikepapadim

pom.xml

@@ -72,8 +72,9 @@
         <build>
             <plugins>
                 <!-- Compiler: enable-preview for both JDKs.
-                     jdk.incubator.vector is added only for JDK21 (see jdk21 profile below).
-                     On JDK25 Vector API is stable; the add-modules flag is not needed. -->
+                     --add-modules jdk.incubator.vector is added per JDK profile (see below).
+                     The Vector API remains in jdk.incubator.vector on both JDK 21 and JDK 25
+                     (JEP 508 — 10th Incubator in JDK 25); the flag is required for all JDKs. -->
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-compiler-plugin</artifactId>
@@ -169,7 +170,8 @@
                  Auto-activates for JDK 25.0.2+ builds (minimum required version).
                  Publishes: gpu-llama3:${revision}-jdk25
                  TornadoVM: 3.0.0-jdk25
-                 Vector API is stable in JDK25; add-modules jdk.incubator.vector not needed
+                 Vector API is still incubating in JDK 25 (JEP 508 — 10th Incubator);
+                 --add-modules jdk.incubator.vector is required for compilation.
             ─────────────────────────────────────────────────────────────────────── -->
             <profile>
                 <id>jdk25</id>
@@ -180,6 +182,21 @@
                     <tornadovm.version>3.0.0-jdk25</tornadovm.version>
                     <jdk.version.suffix>-jdk25</jdk.version.suffix>
                 </properties>
+                <build>
+                    <plugins>
+                        <plugin>
+                            <groupId>org.apache.maven.plugins</groupId>
+                            <artifactId>maven-compiler-plugin</artifactId>
+                            <configuration>
+                                <!-- Appended to enable-preview already in main build -->
+                                <compilerArgs combine.children="append">
+                                    <arg>--add-modules</arg>
+                                    <arg>jdk.incubator.vector</arg>
+                                </compilerArgs>
+                            </configuration>
+                        </plugin>
+                    </plugins>
+                </build>
             </profile>
 
             <profile>
@@ -205,7 +222,10 @@
                             </executions>
                         </plugin>
 
-                        <!-- Javadocs with preview feature support -->
+                        <!-- Javadocs with preview feature support.
+                             --add-modules jdk.incubator.vector is required on all currently
+                             supported JDKs (21 and 25) because the Vector API remains in the
+                             incubator module (JEP 508 — 10th Incubator in JDK 25). -->
                         <plugin>
                             <groupId>org.apache.maven.plugins</groupId>
                             <artifactId>maven-javadoc-plugin</artifactId>
@@ -215,9 +235,12 @@
                                 <release>${maven.compiler.source}</release>
                                 <additionalJOptions>
                                     <additionalJOption>--enable-preview</additionalJOption>
+                                    <additionalJOption>--add-modules</additionalJOption>
+                                    <additionalJOption>jdk.incubator.vector</additionalJOption>
                                 </additionalJOptions>
                                 <additionalOptions>
                                     <additionalOption>--enable-preview</additionalOption>
+                                    <additionalOption>--add-modules jdk.incubator.vector</additionalOption>
                                 </additionalOptions>
                                 <failOnError>false</failOnError>
                                 <failOnWarnings>false</failOnWarnings>