Re: [CHKP] Fix for PR79988

2017-04-20 Thread Ilya Enkovich
Hi, Please put comment to code explaining why you don't use gimple_call_builtin_p to avoid similar issues in the future. Also please follow Rainer's comments. OK with these fixes. Thanks, Ilya 2017-04-20 15:27 GMT+03:00 Rainer Orth : > Hi Alexander, > > just a couple of nits: > >> gcc/testsui

Re: [CHKP] Fix for PR79988

2017-04-20 Thread Rainer Orth
Hi Alexander, just a couple of nits: > gcc/testsuite/ChangeLog: > > * gcc.target/i386/mpx/PR79988.c: New test. We usually don't use capital PR in testcase names. Please use pr79988.c instead, matching the other files there. Also, both ChangeLog entries should include a PR reference lik

[CHKP] Fix for PR79988

2017-04-20 Thread Alexander Ivchenko
Hi Ilya, The patch below is basically what Richard proposed in Bugzilla. This approach should produce the correct code for any address spaces, because it will just strip the address space part of the pointer in all bnd* instructions; Since we will do that consistently, all checks should be consist