[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