On 11/20/2017 02:58 PM, Jim Wilson wrote:
The dwarf2out.c patch looks good to me, though the testcase does not
fail on unpatched mainline anymore. I had to go back to the 2017-10-22
snapshot to see the failure. There was a followup from Mark Wielaard
mentioning that elfutils still fails on mainline, so maybe we can get a
testcase from there.
The testcase was "broken" by Jan's big patch on Nov 3
https://gcc.gnu.org/ml/gcc-patches/2017-11/msg00104.html
It didn't seem useful to look into that, so I tried running
multidelta/creduce on the elfutils sources, and managed to get a small
testcase that still fails with mainline. I then fixed it to get rid of
the warnings.
#include <stdlib.h>
extern int c (void);
int
a(int b) {
if (c())
abort();
}
There is no guarantee that this testcase will still show the problem in
the future, but it seems good enough for now.
OK with this testcase. And with your testcase too if you think that
still has value.
Jim