On Mon, 2020-03-30 at 12:09 -0400, David Malcolm via Gcc-patches wrote: > On Sun, 2020-03-29 at 21:31 +0100, Andrea Corallo wrote: > > David Malcolm <dmalc...@redhat.com> writes: > > > > > On Wed, 2020-03-18 at 23:51 +0100, Andrea Corallo wrote: > > > Please add the new test to the header in its alphabetical > > > location, > > > i.e. between: > > > > > > /* test-vector-types.cc: We don't use this, since it's C++. */ > > > > > > and > > > > > > /* test-volatile.c */ > > > > > > An entry also needs to be added to the "testcases" array at the > > > end > > > of > > > the header (again, in the alphabetical-sorted location). > > > > I tried adding the test into the "testcases" array but this makes > > the > > threads test failing. > > > > I think this has nothing to do with this patch and happen just > > because > > this test does not define any code. Infact I see the same > > happening > > just adding "test-empty.c" to the "testcases" array on the current > > master. > > > > The error is not very reproducible, I tried a run under valgrind > > but > > have found nothing so far :/ > > > > Dave do you recall if there was a specific reason not to have > > "test-empty.c" into the "testcases" array? > > > > Andrea > > I replied to this as: > > [PATCH] lra: set insn_code_data to NULL when freeing > https://gcc.gnu.org/pipermail/gcc-patches/2020-March/542929.html > > but forgot to set the reply-to in git send-email; sorry. >
Andrea: I've pushed my proposed fix for the above to master as 3809bcd6c0ee324cbd855c68cee104c8bf134dbe. Does this fix the issue you were seeing? Thanks Dave