[code-reflection] RFR: ONNX FFM Runtime initial work [v3]
Adam Sotona
asotona at openjdk.org
Mon Feb 10 13:53:26 UTC 2025
On Mon, 10 Feb 2025 13:09:17 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional commit since the last revision:
>>
>> minor rename
>
> cr-examples/onnx/src/main/java/oracle/code/onnx/OnnxRuntime.java line 153:
>
>> 151: }
>> 152:
>> 153: public Environment createEnv(LoggingLevel logLevel, String logId) {
>
> There's some lifetime here to deal with... createEnv has a releaseEnv method, which is not dealt with here. Perhaps it would be convenient to associate the returned env segment with an automatic arena (Arena::ofAuto) that will be kept alive as long as the Environment instance is reachable which is, I think, what you want.
Yes, there are definitely many places for improvements. This is just a first shot to make it working ASAP.
I expect many refactoring rounds.
-------------
PR Review Comment: https://git.openjdk.org/babylon/pull/311#discussion_r1949092878
More information about the babylon-dev
mailing list