Re: RFR: 8329997: Add MemorySegment::reinterpretate overloads for alignment constraints

2024-04-10 Thread Maurizio Cimadamore
On Wed, 10 Apr 2024 12:49:11 GMT, Per Minborg wrote: > This PR proposes to add two overloads `MemorySegment::reinterpretate` to > allow aligned reinterpretation. Should we add the full set of overloads? E.g. reinterpret(arena, cleanup) reinterpret(size) reinterpret(size, arena, cleanup) reinter

Re: RFR: 8329997: Add MemorySegment::reinterpretate overloads for alignment constraints

2024-04-10 Thread Maurizio Cimadamore
On Wed, 10 Apr 2024 15:19:31 GMT, Jorn Vernee wrote: >> This PR proposes to add two overloads `MemorySegment::reinterpretate` to >> allow aligned reinterpretation. > > test/jdk/java/foreign/TestSegments.java line 396: > >> 394: AtomicInteger counter = new AtomicInteger(); >> 395:

Re: RFR: 8329997: Add MemorySegment::reinterpretate overloads for alignment constraints

2024-04-10 Thread Jorn Vernee
On Wed, 10 Apr 2024 12:49:11 GMT, Per Minborg wrote: > This PR proposes to add two overloads `MemorySegment::reinterpretate` to > allow aligned reinterpretation. src/java.base/share/classes/jdk/internal/foreign/SegmentFactories.java line 72: > 70: sessionImpl.addCloseAction(action)

RFR: 8329997: Add MemorySegment::reinterpretate overloads for alignment constraints

2024-04-10 Thread Per Minborg
This PR proposes to add two overloads `MemorySegment::reinterpretate` to allow aligned reinterpretation. - Commit messages: - Update copyright year - Add MemorySegment::reinterpretate overloads for alignment constraints Changes: https://git.openjdk.org/jdk/pull/18717/files Webre