public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH 0/9] Don't add crtfastmath.o for -shared
@ 2023-01-13  8:05 Richard Biener
  2023-01-13  8:08 ` Hongtao Liu
  2023-01-27 12:17 ` Kyrylo Tkachov
  0 siblings, 2 replies; 6+ messages in thread
From: Richard Biener @ 2023-01-13  8:05 UTC (permalink / raw)
  To: gcc-patches; +Cc: hongtao.liu, ubizjak


This is a series completing the fix for PR55522 which got a fix for
x86-linux already but left all other targets unfixed (including
x86-cygwin, x86-darwin and x86-mingw32).  The following series
applies a similar change to other specs using crtfastmath.o,
the changes are untested.

Target maintainers are CCed and I hope they can smoke-test the
changes.

x86 maintainers, can you please adjust the missed specs yourself?

Thanks,
Richard.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/9] Don't add crtfastmath.o for -shared
  2023-01-13  8:05 [PATCH 0/9] Don't add crtfastmath.o for -shared Richard Biener
@ 2023-01-13  8:08 ` Hongtao Liu
  2023-01-27 12:17 ` Kyrylo Tkachov
  1 sibling, 0 replies; 6+ messages in thread
From: Hongtao Liu @ 2023-01-13  8:08 UTC (permalink / raw)
  To: Richard Biener; +Cc: gcc-patches, hongtao.liu, ubizjak

On Fri, Jan 13, 2023 at 4:05 PM Richard Biener via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
>
> This is a series completing the fix for PR55522 which got a fix for
> x86-linux already but left all other targets unfixed (including
> x86-cygwin, x86-darwin and x86-mingw32).  The following series
> applies a similar change to other specs using crtfastmath.o,
> the changes are untested.
>
> Target maintainers are CCed and I hope they can smoke-test the
> changes.
>
> x86 maintainers, can you please adjust the missed specs yourself?
Sure, Let me do this.
>
> Thanks,
> Richard.



-- 
BR,
Hongtao

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH 0/9] Don't add crtfastmath.o for -shared
  2023-01-13  8:05 [PATCH 0/9] Don't add crtfastmath.o for -shared Richard Biener
  2023-01-13  8:08 ` Hongtao Liu
@ 2023-01-27 12:17 ` Kyrylo Tkachov
  2023-01-27 12:24   ` Richard Biener
  1 sibling, 1 reply; 6+ messages in thread
From: Kyrylo Tkachov @ 2023-01-27 12:17 UTC (permalink / raw)
  To: Richard Biener, gcc-patches; +Cc: hongtao.liu, ubizjak

Thanks for fixing this Richard.

> -----Original Message-----
> From: Gcc-patches <gcc-patches-
> bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Richard
> Biener via Gcc-patches
> Sent: Friday, January 13, 2023 8:05 AM
> To: gcc-patches@gcc.gnu.org
> Cc: hongtao.liu@intel.com; ubizjak@gmail.com
> Subject: [PATCH 0/9] Don't add crtfastmath.o for -shared
> 
> 
> This is a series completing the fix for PR55522 which got a fix for
> x86-linux already but left all other targets unfixed (including
> x86-cygwin, x86-darwin and x86-mingw32).  The following series
> applies a similar change to other specs using crtfastmath.o,
> the changes are untested.
> 
> Target maintainers are CCed and I hope they can smoke-test the
> changes.
> 

Do you think it's something we should mention in changes.html for GCC 13?
Kyrill

> x86 maintainers, can you please adjust the missed specs yourself?
> 
> Thanks,
> Richard.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH 0/9] Don't add crtfastmath.o for -shared
  2023-01-27 12:17 ` Kyrylo Tkachov
@ 2023-01-27 12:24   ` Richard Biener
  2023-01-27 12:27     ` Richard Sandiford
  0 siblings, 1 reply; 6+ messages in thread
From: Richard Biener @ 2023-01-27 12:24 UTC (permalink / raw)
  To: Kyrylo Tkachov; +Cc: gcc-patches, hongtao.liu, ubizjak

On Fri, 27 Jan 2023, Kyrylo Tkachov wrote:

> Thanks for fixing this Richard.
> 
> > -----Original Message-----
> > From: Gcc-patches <gcc-patches-
> > bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Richard
> > Biener via Gcc-patches
> > Sent: Friday, January 13, 2023 8:05 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: hongtao.liu@intel.com; ubizjak@gmail.com
> > Subject: [PATCH 0/9] Don't add crtfastmath.o for -shared
> > 
> > 
> > This is a series completing the fix for PR55522 which got a fix for
> > x86-linux already but left all other targets unfixed (including
> > x86-cygwin, x86-darwin and x86-mingw32).  The following series
> > applies a similar change to other specs using crtfastmath.o,
> > the changes are untested.
> > 
> > Target maintainers are CCed and I hope they can smoke-test the
> > changes.
> > 
> 
> Do you think it's something we should mention in changes.html for GCC 13?

Sure, I will add something once the rest of the series is approved.

Richard.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/9] Don't add crtfastmath.o for -shared
  2023-01-27 12:24   ` Richard Biener
@ 2023-01-27 12:27     ` Richard Sandiford
  2023-01-27 14:52       ` Richard Biener
  0 siblings, 1 reply; 6+ messages in thread
From: Richard Sandiford @ 2023-01-27 12:27 UTC (permalink / raw)
  To: Richard Biener via Gcc-patches
  Cc: Kyrylo Tkachov, Richard Biener, hongtao.liu, ubizjak

Richard Biener via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> On Fri, 27 Jan 2023, Kyrylo Tkachov wrote:
>
>> Thanks for fixing this Richard.
>> 
>> > -----Original Message-----
>> > From: Gcc-patches <gcc-patches-
>> > bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Richard
>> > Biener via Gcc-patches
>> > Sent: Friday, January 13, 2023 8:05 AM
>> > To: gcc-patches@gcc.gnu.org
>> > Cc: hongtao.liu@intel.com; ubizjak@gmail.com
>> > Subject: [PATCH 0/9] Don't add crtfastmath.o for -shared
>> > 
>> > 
>> > This is a series completing the fix for PR55522 which got a fix for
>> > x86-linux already but left all other targets unfixed (including
>> > x86-cygwin, x86-darwin and x86-mingw32).  The following series
>> > applies a similar change to other specs using crtfastmath.o,
>> > the changes are untested.
>> > 
>> > Target maintainers are CCed and I hope they can smoke-test the
>> > changes.
>> > 
>> 
>> Do you think it's something we should mention in changes.html for GCC 13?
>
> Sure, I will add something once the rest of the series is approved.

Mind if I rubber-stamp OK the unreviewed changes?  I don't think there's
a good justification for making a different choice on different targets.

Thanks,
Richard

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/9] Don't add crtfastmath.o for -shared
  2023-01-27 12:27     ` Richard Sandiford
@ 2023-01-27 14:52       ` Richard Biener
  0 siblings, 0 replies; 6+ messages in thread
From: Richard Biener @ 2023-01-27 14:52 UTC (permalink / raw)
  To: Richard Sandiford
  Cc: Richard Biener via Gcc-patches, Kyrylo Tkachov, hongtao.liu, ubizjak

On Fri, 27 Jan 2023, Richard Sandiford wrote:

> Richard Biener via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> > On Fri, 27 Jan 2023, Kyrylo Tkachov wrote:
> >
> >> Thanks for fixing this Richard.
> >> 
> >> > -----Original Message-----
> >> > From: Gcc-patches <gcc-patches-
> >> > bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Richard
> >> > Biener via Gcc-patches
> >> > Sent: Friday, January 13, 2023 8:05 AM
> >> > To: gcc-patches@gcc.gnu.org
> >> > Cc: hongtao.liu@intel.com; ubizjak@gmail.com
> >> > Subject: [PATCH 0/9] Don't add crtfastmath.o for -shared
> >> > 
> >> > 
> >> > This is a series completing the fix for PR55522 which got a fix for
> >> > x86-linux already but left all other targets unfixed (including
> >> > x86-cygwin, x86-darwin and x86-mingw32).  The following series
> >> > applies a similar change to other specs using crtfastmath.o,
> >> > the changes are untested.
> >> > 
> >> > Target maintainers are CCed and I hope they can smoke-test the
> >> > changes.
> >> > 
> >> 
> >> Do you think it's something we should mention in changes.html for GCC 13?
> >
> > Sure, I will add something once the rest of the series is approved.
> 
> Mind if I rubber-stamp OK the unreviewed changes?  I don't think there's
> a good justification for making a different choice on different targets.

Sure, I've pushed the rest of the changes now.  I'm installing below
to changes.html.

Richard.

diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html
index 6cd5dd64..9ecd115c 100644
--- a/htdocs/gcc-13/changes.html
+++ b/htdocs/gcc-13/changes.html
@@ -123,6 +123,10 @@ a work-in-progress.</p>
             system overcommitting.</li>
     </ul>
   </li>
+  <li><code>-Ofast</code>, <code>-ffast-math</code> and <code>-funsafe-math-optimizations</code>
+      will no longer add startup code to alter the floating-point environment
+      when producing a shared object with <code>-shared</code>.
+  </li>
 </ul>
 
 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-01-27 14:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-13  8:05 [PATCH 0/9] Don't add crtfastmath.o for -shared Richard Biener
2023-01-13  8:08 ` Hongtao Liu
2023-01-27 12:17 ` Kyrylo Tkachov
2023-01-27 12:24   ` Richard Biener
2023-01-27 12:27     ` Richard Sandiford
2023-01-27 14:52       ` Richard Biener

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).