[foreign-memaccess+abi] RFR: 8313238: Improve the performance of strlen for null-terminated strings

ExE Boss duke at openjdk.org
Thu Aug 3 17:33:47 UTC 2023


On Thu, 3 Aug 2023 13:52:27 GMT, Per Minborg <pminborg at openjdk.org> wrote:

> This PR proposes to use native calls for determining the string length for C-type strings. 
> 
> Smaller segments (<1024 bytes) is using a trivial call whereas all other segments are using a normal call. We might consider always using regular calls if we think most segments are unbound anyhow.
> 
> The PR also contains a number of new tests, one of which requires a large heap size to run. It is likely that this latter test will not run in most test environments. In such cases, it is silently ignored.

src/java.base/share/classes/jdk/internal/foreign/StringSupport.java line 132:

> 130:             }
> 131:             return (int)len;
> 132:         } catch (Throwable e) {

This should rethrow `Error`s and `RuntimeException`s:
Suggestion:

        } catch (RuntimeException | Error e) {
            throw e;
        } catch (Throwable e) {

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

PR Review Comment: https://git.openjdk.org/panama-foreign/pull/857#discussion_r1283510839


More information about the panama-dev mailing list