Hi Andre, > Because this patch is obvious I plan to commit it tomorrow if no one objects?!
The patch itself is obviously OK. About the test case: In general, it is better not to change existing test cases unless absolutely necessary (e.g. adjust an error message). This makes it easier to track regressions. I would prefer if you made a new test case from your existing one, with the changes you did and a small explanation of what was tested in the comments. If you are worried about runtime for an additonal test, you can use the ! { dg-do run } hack (notice the two spaces between the dg-do and the run) to have the test case execute only once. OK with that change. Regards Thomas