Re: [PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

2015-02-25 Thread Uros Bizjak
On Wed, Feb 25, 2015 at 3:47 PM, Uros Bizjak ubiz...@gmail.com wrote:
 Hello!

 2015-02-25  Ilya Enkovich  ilya.enkov...@intel.com

 PR target/65167
 * gcc/config/i386/i386.c (ix86_function_arg_regno_p): Support
 bounds registers.
 (avoid_func_arg_motion): Add dependencies for BNDSTX insns.

 gcc/testsuite/

 2015-02-25  Ilya Enkovich  ilya.enkov...@intel.com

 PR target/65167
 * gcc.target/i386/pr65167.c: New.

 OK for mainline with a small change below.

Ah, I missed the explanation w.r.t __builtin_apply_args in the PR.

The patch is OK as it is.

Thanks,
Uros.


Re: [PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

2015-02-25 Thread Uros Bizjak
Hello!

 2015-02-25  Ilya Enkovich  ilya.enkov...@intel.com

 PR target/65167
 * gcc/config/i386/i386.c (ix86_function_arg_regno_p): Support
 bounds registers.
 (avoid_func_arg_motion): Add dependencies for BNDSTX insns.

 gcc/testsuite/

 2015-02-25  Ilya Enkovich  ilya.enkov...@intel.com

 PR target/65167
 * gcc.target/i386/pr65167.c: New.

OK for mainline with a small change below.

+  if (TARGET_MPX  BND_REGNO_P (regno))
+return true;

The check for TARGET_MPX is not needed.

Uros.