On 09/12/2017 02:29 PM, Aldy Hernandez wrote: > I'm going to abuse either my *gimpl* maintainer hat or my obvious hat > and approve this. > > However, since we already have another test that looks the same > (gimplefe-error-2.c) but without the 0 case, it would probably be best > to add a comment in gimplefe-14.c as to why there is a 0 case. In the > case of gimplefe-error-2.c, it doesn't matter because it is only a > compile test. > > OK with that nit.
Thanks for the note, installed as r252024. Martin > > > On Tue, Sep 12, 2017 at 3:44 AM, Martin Liška <mli...@suse.cz> wrote: >> Hello. >> >> Simple fix for args == 0 also returns 0 return value. >> >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> >> Ready to be installed? >> Martin >> >> gcc/testsuite/ChangeLog: >> >> 2017-09-11 Martin Liska <mli...@suse.cz> >> >> PR testsuite/82114 >> * gcc.dg/gimplefe-14.c (main): Add handling of case 0. >> --- >> gcc/testsuite/gcc.dg/gimplefe-14.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >>
