Hello all,
On 11/02/2016 05:33 PM, Jakub Jelinek wrote:
On Wed, Nov 02, 2016 at 04:44:05PM +0100, Jakub Jelinek wrote:
You could find an Ada test that uses the code and verify that the
output stays the same?
I can try to find the patch that introduced it and if it contained any
testcases.
I couldn't find any unfortunately. Pierre, could you please test if the
following patch doesn't regress anything in the Ada debug info area?
Yes, DWARF output is hard to test… or at least I haven’t find a way to
do so in TCL/DejaGNU. ;-)
I have my own Python-based testsuite, which I use everytime I submit a
DIE-related patch. As I said on IRC and as richi and I discussed at the
last cauldron, the plan would be to manage to port the testcase to
GCC’s, but schedule-wise I’m not able to do that these days… For the
record I still plan to do it, though! So, just in case someone would
need/want to run this testsuite, here it is in its current state:
<https://github.com/pmderodat/dwarf-ada-testsuite>.
Anyway, that being said: your patch does not introduce regressions in my
testsuite, so it’s fine as far as I’m concerned. :-) Thank you for asking!
--
Pierre-Marie de Rodat
