This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] fix for PR 59691

From: Jeff Law <law at redhat dot com>
To: "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: "'H.J. Lu (hjl dot tools at gmail dot com)'" <hjl dot tools at gmail dot com>, "bernd dot edlinger at hotmail dot de" <bernd dot edlinger at hotmail dot de>, "pinskia at gcc dot gnu dot org" <pinskia at gcc dot gnu dot org>
Date: Mon, 10 Feb 2014 09:41:45 -0700
Subject: Re: [PATCH] fix for PR 59691
Authentication-results: sourceware.org; auth=none
References: <BF230D13CA30DD48930C31D4099330003A4CCA39 at FMSMSX101 dot amr dot corp dot intel dot com>

On 02/06/14 15:21, Iyer, Balaji V wrote:

Hello Everyone,
	Attached, please find a patch that will fix the issue in PR 59691. The main issue was that the Cilk library (libcilkrts) was not checking if the target has SSE support before emitting SSE instruction. This patch should fix that.

Here is the ChangeLog entries:
libcilkrts/ChangeLog
2014-02-06  Balaji V. Iyer  <balaji.v.iyer@intel.com>

         * runtime/config/x86/os-unix-sysdep.c (__builtin_cpu_supports): New
         function.
         (restore_x86_fp_state): Added a check if the cpu supports the
         instruction before emitting it.
         (sysdep_save_fp_ctrl_state): Likewise.



Is this Ok to install?

Yes, this is fine.

jeff

References:
- [PATCH] fix for PR 59691
  - From: Iyer, Balaji V

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]