Re: RFR: 8320500: [vectorapi] RISC-V: Optimize vector math operations with SLEEF [v8]

2024-09-27 Thread Hamlin Li
On Fri, 27 Sep 2024 02:22:58 GMT, Fei Yang wrote: >> Hamlin Li has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix test macro > > src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 51: > >> 49: // the dynamic roun

RFR: 8341024: [test] build/AbsPathsInImage.java fails with OOM when using ubsan-enabled binaries

2024-09-27 Thread Matthias Baesken
The jtreg test build/AbsPathsInImage.java fails with OOM when using ubsan-enabled binaries (on Linux x86_64). Reason seems to be that the ubsan-enabled binaries are much larger than 'normal' product binaries. (for debug binaries the test is already disabled) Error is : java.lang.OutOfMemoryError:

Re: RFR: 8320500: [vectorapi] RISC-V: Optimize vector math operations with SLEEF [v8]

2024-09-27 Thread Fei Yang
On Fri, 27 Sep 2024 07:06:44 GMT, Hamlin Li wrote: >> src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 51: >> >>> 49: // the dynamic rounding mode is always RNE. >>> 50: >>> 51: #ifdef DEBUG >> >> Question: Should we check for `NDEBUG` macro (A macro specified by C/C++

Re: RFR: 8341024: [test] build/AbsPathsInImage.java fails with OOM when using ubsan-enabled binaries

2024-09-27 Thread Erik Joelsson
On Fri, 27 Sep 2024 10:05:08 GMT, Matthias Baesken wrote: > The jtreg test build/AbsPathsInImage.java fails with OOM when using > ubsan-enabled binaries (on Linux x86_64). > Reason seems to be that the ubsan-enabled binaries are much larger than > 'normal' product binaries. > (for debug binarie

Re: RFR: 8320500: [vectorapi] RISC-V: Optimize vector math operations with SLEEF [v8]

2024-09-27 Thread Hamlin Li
On Fri, 27 Sep 2024 08:10:35 GMT, Fei Yang wrote: >> `NDEBUG` is only used in sleefdp.c which is the original sleef code, and we >> don't use that file in jdk directly, in java.base module of jdk it uses >> `DEBUG` but not use `NDEBUG`. >> Based on above information, I think `DEBUG` is better.

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v28]

2024-09-27 Thread Roman Kennke
> This is the main body of the JEP 450: Compact Object Headers (Experimental). > > It is also a follow-up to #20640, which now also includes (and supersedes) > #20603 and #20605, plus the Tiny Class-Pointers parts that have been > previously missing. > > Main changes: > - Introduction of the (

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v28]

2024-09-27 Thread Roberto CastaƱeda Lozano
On Thu, 12 Sep 2024 15:42:59 GMT, Thomas Stuefe wrote: >> src/hotspot/share/opto/machnode.cpp line 390: >> >>> 388: t = t->make_ptr(); >>> 389: } >>> 390: if (t->isa_narrowklass() && CompressedKlassPointers::shift() == 0) { >> >> Does this change have any effect? `UseCompressedClassPoin

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v26]

2024-09-27 Thread Scott Gibbons
On Fri, 27 Sep 2024 08:24:50 GMT, Roman Kennke wrote: >> @rkennke I reviewed [rkennke@ >> 097c2af](https://github.com/rkennke/jdk/commit/097c2afa04397773e514552dfb942aa889bfa2c1) >> and the code looks good to me. I would prefer this approach instead of not >> generating the IndexOf intrinsic.

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v26]

2024-09-27 Thread Roman Kennke
On Thu, 26 Sep 2024 17:25:06 GMT, Scott Gibbons wrote: >> @sviswa7 or @asgibbons WDYT about including >> https://github.com/rkennke/jdk/commit/097c2afa04397773e514552dfb942aa889bfa2c1 >> as part of compact object headers implementation? Otherwise we would have >> to disable indexOf intrinsic w

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v26]

2024-09-27 Thread Roman Kennke
On Fri, 27 Sep 2024 14:44:35 GMT, Scott Gibbons wrote: >> I like to have the functional connection: if - for whatever reason - the >> array base offset is smaller than 16, we need to deal with that. The reason >> for this happens to be `UseCompactObjectHeaders`, but that may not be clear >> to

Re: RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v9]

2024-09-27 Thread Yudi Zheng
On Thu, 19 Sep 2024 14:22:51 GMT, Stefan Karlsson wrote: >> We haven't decided whether or not we will git rid of >> ```Klass::_prototype_header``` before intergrating this PR, or not. @stefank >> could point you to a WIP branch, if that's helpful. > > This is my current work-in-progress code: >