On Thu, 7 May 2026 21:39:14 GMT, Coleen Phillimore <[email protected]> wrote:

>> Add a bunch of stubs and parameter changes to get non-Oracle platforms to 
>> compile.  Tested with cross compilation in mach5.  Also enabled GHA for 
>> these platforms.  Tested tier1 with Oracle platforms.
>> 
>> Porters please let me know if this runs hello world, then the rest is up to 
>> you.
>> Thank you!
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Coleen Phillimore has updated the pull request incrementally with two 
> additional commits since the last revision:
> 
>  - include another file.
>  - Some forward declarations might do the trick.

src/hotspot/cpu/riscv/riscv.ad line 1788:

> 1786: #ifndef PRODUCT
> 1787: void MachVEPNode::format(PhaseRegAlloc* ra_, outputStream* st) const
> 1788: {

Suggestion:

{
  Unimplemented();

src/hotspot/cpu/riscv/riscv.ad line 1793:

> 1791: 
> 1792: void MachVEPNode::emit(C2_MacroAssembler *masm, PhaseRegAlloc* ra_) 
> const
> 1793: {

Suggestion:

{
  Unimplemented();

src/hotspot/cpu/s390/s390.ad line 1610:

> 1608: #ifndef PRODUCT
> 1609: void MachVEPNode::format(PhaseRegAlloc* ra_, outputStream* st) const
> 1610: {

Suggestion:

{
  Unimplemented();

src/hotspot/cpu/s390/s390.ad line 1615:

> 1613: 
> 1614: void MachVEPNode::emit(C2_MacroAssembler *masm, PhaseRegAlloc* ra_) 
> const
> 1615: {

Suggestion:

{
  Unimplemented();

-------------

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2396#discussion_r3206834579
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2396#discussion_r3206835282
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2396#discussion_r3206842395
PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2396#discussion_r3206843257

Reply via email to