On Wed, 26 Jan 2022 14:23:14 GMT, Erik Joelsson <er...@openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request incrementally with two 
>> additional commits since the last revision:
>> 
>>  - Update test/hotspot/jtreg/runtime/ErrorHandling/TestDwarf.java
>>    
>>    Co-authored-by: Erik Joelsson <37597443+erik...@users.noreply.github.com>
>>  - Update test/hotspot/jtreg/runtime/ErrorHandling/TestDwarf.java
>>    
>>    Co-authored-by: Erik Joelsson <37597443+erik...@users.noreply.github.com>
>
> Build changes look good.

I've pushed an update to fix the builds, add some assertions and to add the 
`gdwarf-4` GCC flag for now to exclude DWARF 5 and let the tests pass again 
(@erikj79 is that the right place? Do I need to add it at other places as 
well?). 

I think this `gdwarf-4` flag is currently the only option to eventually move 
forward with this RFE without extending the parser to support DWARF 5. I could 
still do that as a follow-up task and then remove the `gdwarf-4` flag again. 
But also adding DWARF 5 support in this patch is probably too much. What do you 
think about that?

As mentioned above, I'm going to be away for the rest of the month. I will get 
back to this PR at the start of May again.

Cheers,
Christian

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

PR: https://git.openjdk.java.net/jdk/pull/7126

Reply via email to