| Message ID | 20161229152642.GQ21933@tucnak |
|---|---|
| State | New |
| Headers | show |
On Thu, Dec 29, 2016 at 4:26 PM, Jakub Jelinek <jakub@redhat.com> wrote: > On Tue, Dec 27, 2016 at 06:36:11PM +0300, Alexander Ivchenko wrote: >> Committed as r243942 with the ChangeLog entries > > Unfortunately it fails if assembler has mpx support, but hw doesn't support > it. > > The following patch should fix that. Tested on x86_64-linux, ok for trunk? > > 2016-12-29 Jakub Jelinek <jakub@redhat.com> > > * gcc.target/i386/mpx/memcpy-1.c: Include mpx-check.h. > (main): Renamed to ... > (mpx_test): ... this. Add argc and argv arguments. OK. Thanks, Uros. > --- gcc/testsuite/gcc.target/i386/mpx/memcpy-1.c.jj 2016-12-28 13:14:24.000000000 +0100 > +++ gcc/testsuite/gcc.target/i386/mpx/memcpy-1.c 2016-12-29 16:07:11.135200098 +0100 > @@ -8,6 +8,7 @@ > > #include <stdio.h> > #include <string.h> > +#include "mpx-check.h" > > char s[10]; > char d[10]; > @@ -16,7 +17,7 @@ __attribute__((noinline)) > char* foo(char* dst, char* src, size_t size) { > return memcpy(dst, src, size); > } > -int main() { > +int mpx_test(int argc, const char **argv) { > char* r = foo(d, s, 11); > printf("r = %p\n", r); > return 0; > > > Jakub
--- gcc/testsuite/gcc.target/i386/mpx/memcpy-1.c.jj 2016-12-28 13:14:24.000000000 +0100 +++ gcc/testsuite/gcc.target/i386/mpx/memcpy-1.c 2016-12-29 16:07:11.135200098 +0100 @@ -8,6 +8,7 @@ #include <stdio.h> #include <string.h> +#include "mpx-check.h" char s[10]; char d[10]; @@ -16,7 +17,7 @@ __attribute__((noinline)) char* foo(char* dst, char* src, size_t size) { return memcpy(dst, src, size); } -int main() { +int mpx_test(int argc, const char **argv) { char* r = foo(d, s, 11); printf("r = %p\n", r); return 0;
On Tue, Dec 27, 2016 at 06:36:11PM +0300, Alexander Ivchenko wrote: > Committed as r243942 with the ChangeLog entries Unfortunately it fails if assembler has mpx support, but hw doesn't support it. The following patch should fix that. Tested on x86_64-linux, ok for trunk? 2016-12-29 Jakub Jelinek <jakub@redhat.com> * gcc.target/i386/mpx/memcpy-1.c: Include mpx-check.h. (main): Renamed to ... (mpx_test): ... this. Add argc and argv arguments. Jakub