Re: [PATCH] i386: Make xmm16-xmm31 call used even in ms ABI

2020-02-05 Thread JonY
On 2/4/20 11:42 AM, Jakub Jelinek wrote: > Hi! > > On Tue, Feb 04, 2020 at 11:16:06AM +0100, Uros Bizjak wrote: >> I guess that Comment #9 patch form the PR should be trivially correct, >> but althouhg it looks obvious, I don't want to propose the patch since >> I have no means of testing it. > >

Re: [PATCH] x86-64: Pass aggregates with only float/double in GPRs for MS_ABI

2020-02-07 Thread JonY
On 2/7/20 3:23 AM, H.J. Lu wrote: > On Wed, Feb 05, 2020 at 09:51:14PM +0100, Uros Bizjak wrote: >> On Wed, Feb 5, 2020 at 6:59 PM H.J. Lu wrote: >>> >>> MS_ABI requires passing aggregates with only float/double in integer >>> registers. Checked gcc outputs against Clang and fixed: >>> >>> FAIL:

Re: [PATCH] i386: Make xmm16-xmm31 call used even in ms ABI

2020-02-07 Thread JonY
On 2/6/20 6:07 AM, Jakub Jelinek wrote: > On Thu, Feb 06, 2020 at 01:00:36AM +0000, JonY wrote: >> On 2/4/20 11:42 AM, Jakub Jelinek wrote: >>> Hi! >>> >>> On Tue, Feb 04, 2020 at 11:16:06AM +0100, Uros Bizjak wrote: >>>> I guess that Comment

Re: [PATCH] i386: Make xmm16-xmm31 call used even in ms ABI

2020-02-08 Thread JonY
On 2/7/20 11:28 AM, Jakub Jelinek wrote: > On Fri, Feb 07, 2020 at 10:57:22AM +0000, JonY wrote: >>>> Is this patch testing still required? I just got back from traveling. >>> >>> Yes, our reading of the MS ABI docs show that xmm16-31 are to be call used >>

[Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from

2012-10-28 Thread JonY
Hi, Workaround now exists for trunk mingw-w64 headers. Kai approved over IRC, so anybody with commit rights please push. ChangeLog 2012-10-29 Jonathan Yong * config/os/mingw32-w64/os_defines.h: Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF as no longer required. Index: libstdc++-v

Re: [Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from mingw32-w64/os_defines.h

2012-10-29 Thread JonY
Fix the mail subject line. On 10/29/2012 06:56, JonY wrote: > Hi, > > Workaround now exists for trunk mingw-w64 headers. > Kai approved over IRC, so anybody with commit rights please push. > > ChangeLog > 2012-10-29 Jonathan Yong > > * config/os/mingw

Re: [Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from mingw32-w64/os_defines.h

2012-10-30 Thread JonY
On 10/29/2012 21:05, JonY wrote: >> ChangeLog >> 2012-10-29 Jonathan Yong >> >> * config/os/mingw32-w64/os_defines.h: Remove >> _GLIBCXX_HAVE_BROKEN_VSWPRINTF >> as no longer required. >> >> >> >

Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon

2012-10-31 Thread JonY
On 10/30/2012 01:56, Jakub Jelinek wrote: > Status > == > > I'd like to close the stage 1 phase of GCC 4.8 development > on Monday, November 5th. If you have still patches for new features you'd > like to see in GCC 4.8, please post them for review soon. Patches > posted before the freeze, b

Re: [Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from (was Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon)

2012-10-31 Thread JonY
On 10/31/2012 19:12, Jonathan Wakely wrote: > On 31 October 2012 11:01, Jonathan Wakely wrote: >> On 31 October 2012 10:25, JonY wrote: >>> On 10/30/2012 01:56, Jakub Jelinek wrote: >>>> Status >>>> == >>>> >>>> I'

Re: [Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from (was Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon)

2012-10-31 Thread JonY
On 10/31/2012 19:23, JonY wrote: > >> Why is the define commented out by the patch, not simply removed? >> If it's not needed then it's not needed. We have subversion to track >> change history, we don't need to leave dead code lying around with >> com

Re: [Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from (was Re: GCC 4.8.0 Status Report (2012-10-29), Stage 1 to end soon)

2012-10-31 Thread JonY
On 10/31/2012 20:01, Jonathan Wakely wrote: > On 31 October 2012 11:23, JonY wrote: >> On 10/31/2012 19:12, Jonathan Wakely wrote: >>> >>> It looks like the workaround is in mingw not in GCC, so is it a >>> problem that it won't be possible to use GCC 4.8

Revert gcc r227962

2016-02-26 Thread JonY
Hi, I've submitted a patch that was committed as r227962, it causes some unintended side effects (namely libuuid on Cygwin). Can someone please revert? Kai still needs some time to setup his gcc development environment. Thanks. signature.asc Description: OpenPGP digital signature

Re: Revert gcc r227962

2016-02-27 Thread JonY
On 2/27/2016 05:26, Jeff Law wrote: > On 02/26/2016 04:04 AM, JonY wrote: >> Hi, >> >> I've submitted a patch that was committed as r227962, it causes some >> unintended side effects (namely libuuid on Cygwin). Can someone please >> revert? >> &

[PATCH] Fix Cygwin bootstrap failing to find win32 libraries

2015-09-16 Thread JonY
libgcc is failing to find kerne32 etc during the 2nd stage when bootstraping, explicitly add w32api directory to search path. Patch OK? diff --git a/gcc/config/i386/cygwin.h b/gcc/config/i386/cygwin.h index 2a2a0bf..fd3bc0a 100644 --- a/gcc/config/i386/cygwin.h +++ b/gcc/config/i386/cygwin.h @@ -

Re: [PATCH] Fix Cygwin bootstrap failing to find win32 libraries

2015-09-19 Thread JonY
On 9/20/2015 02:52, Kai Tietz wrote: > Hello JonY, > > patch is ok with proper ChangeLog. > > Thanks, > Kai > Am 16.09.2015 13:42 schrieb "JonY" <10wa...@gmail.com>: > Explicitly search sysroot/usr/lib/32api for additional win32 libraries, fixes fa

Re: [PATCH] Fix Cygwin bootstrap failing to find win32 libraries

2015-09-20 Thread JonY
On 9/21/2015 02:05, Kai Tietz wrote: > 2015-09-16 13:42 GMT+02:00 JonY <10wa...@gmail.com>: >> libgcc is failing to find kerne32 etc during the 2nd stage when >> bootstraping, explicitly add w32api directory to search path. >> >> Patch OK? >> >> diff

Re: Two fixes for pretty-print.c for mingw-w64

2018-08-14 Thread JonY
On 08/14/2018 05:55 AM, Liu Hao wrote: > 在 2018/8/14 13:54, Liu Hao 写道: >> The two patches attached have addressed two issues in the ANSI escape >> sequence translator I sent before.  Please review, and consider >> backporting these to gcc-8-branch. >> >> >> >> > > And here are SVN changelogs for

Re: [PR c++/87137] GCC-8 Fix

2018-08-30 Thread JonY
On 08/30/2018 11:59 AM, Nathan Sidwell wrote: > On 08/29/2018 11:06 PM, Liu Hao wrote: > >> It is strictly an ABI break but I doubt whether code in real world >> that got broken by this bug ever exists. Usually when people expect >> consecutive bitfields to be packed into a single word they wouldn

Re: [testsuite, i386] Always check for target i?86 and x86_64

2017-06-20 Thread JonY
On 06/20/2017 01:01 PM, Rainer Orth wrote: > > once I got the syntax right, this worked fine: it needs > > { target { ilp32 || lp64 } } > > instead ;-) > > I've also now managed to complete a Darwin/x86_64 bootstrap by locally > reverting the two culprit patches for PR bootstrap/81033 and

Re: [testsuite, i386] Always check for target i?86 and x86_64

2017-06-20 Thread JonY
On 06/20/2017 01:35 PM, Rainer Orth wrote: > JonY <10wa...@gmail.com> writes: > >> On 06/20/2017 01:01 PM, Rainer Orth wrote: >>> Given that there were no other comments, I've installed the patch. It >>> would still be nice if the Cygwin/MingW mai

Re: [PING] Plugin support on Windows/MinGW

2017-11-22 Thread JonY
On 11/21/2017 07:03 AM, Boris Kolpackov wrote: > Hi, > > I would like to ping this patch: > > https://gcc.gnu.org/ml/gcc-patches/2017-11/msg01040.html > > The changes are fairly conservative: they do not touch much of the > existing module implementation and plugin support on MinGW is disabled >

Re: [PING] Plugin support on Windows/MinGW

2017-11-23 Thread JonY
On 11/22/2017 11:14 AM, Boris Kolpackov wrote: > JonY <10wa...@gmail.com> writes: > >> Is there a problem with using .so for internal libraries instead of >> "dll"... > > I think not but I haven't tested it. The problem with using .so instead

Re: [PING] Plugin support on Windows/MinGW

2017-11-26 Thread JonY
On 11/23/2017 11:34 AM, JonY wrote: > On 11/22/2017 11:14 AM, Boris Kolpackov wrote: >> JonY <10wa...@gmail.com> writes: >> >>> Is there a problem with using .so for internal libraries instead of >>> "dll"... >> >> I think not but I hav

Re: [Patch, Fortran] PR25829: Asynchronous I/O

2018-06-04 Thread JonY
On 06/03/2018 06:59 PM, Nicolas Koenig wrote: > Hello everyone, > > this patch adds asynchronous I/O support. Thomas and I finally finished > a feature-complete and debugged version, so here it is. In order to use > asynchronous I/O, it is still necessary to link against libpthread, > libgomp or a

Re: [Patch, Fortran] PR25829: Asynchronous I/O

2018-06-06 Thread JonY
On 06/04/2018 11:21 PM, JonY wrote: > On 06/03/2018 06:59 PM, Nicolas Koenig wrote: >> Hello everyone, >> >> this patch adds asynchronous I/O support. Thomas and I finally finished >> a feature-complete and debugged version, so here it is. In order to use >> asynch

Re: [PATCH] Fix ms_struct/-mms-bitfields structure layout (PR target/52991)

2018-02-28 Thread JonY
On 02/28/2018 12:26 AM, Jakub Jelinek wrote: > Hi! > > The following patch fixes the reported ms_struct/-mms-bitfields structure > layout issues from PR52991. > > There are multiple issues, two of them introduced by the > https://gcc.gnu.org/ml/gcc-patches/2006-04/msg01064.html -mms-bitfields > r

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-06 Thread JonY
On 10/05/2018 02:18 AM, Alexandre Oliva wrote: > Add a configure knob for mingw32 and 64 toolchains to default passing > --large-address-aware to the linker, when creating 32-bit binaries. > -Wl,--disable-large-address-aware can still reverse its effects. > > I've tested this with cross i686-pc-mi

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-09 Thread JonY
On 10/09/2018 04:59 AM, Alexandre Oliva wrote: > On Oct 5, 2018, Joseph Myers wrote: > >> A new configure option needs documenting in install.texi. > > Ah, yes, thanks for the reminder. > > On Oct 6, 2018, JonY <10wa...@gmail.com> wrote: > >> They

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-09 Thread JonY
On 10/10/2018 03:24 AM, Alexandre Oliva wrote: > On Oct 9, 2018, JonY <10wa...@gmail.com> wrote: > >> It is fine to turn it on by default in 32bit MinGW (i686-*-mingw*), but >> leave the defaults as is for others like Cygwin, I am not too sure of >> the effects f

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-10 Thread JonY
On 10/10/2018 04:58 AM, Alexandre Oliva wrote: > On Oct 10, 2018, JonY <10wa...@gmail.com> wrote: > >> On 10/10/2018 03:24 AM, Alexandre Oliva wrote: >>> On Oct 9, 2018, JonY <10wa...@gmail.com> wrote: > >>> Now, if you wish it to affect Cygwin

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-11 Thread JonY
On 10/11/2018 02:57 AM, NightStrike wrote: > > Except that options typically don't get removed, just deprecated. It > seems cleaner to me to drop mingw from the name and make it default to > enabled for cygwin. > It is already enabled for Cygwin, but good point, --enable-large-address-aware it

Re: [Patch] Bug 88521 - gcc 9.0 from r266355 miscompile x265 for mingw-w64 target

2018-12-21 Thread JonY
On 12/21/18 9:08 AM, Uros Bizjak wrote: > On Thu, Dec 20, 2018 at 1:09 PM Jakub Jelinek wrote: >> >> On Thu, Dec 20, 2018 at 01:42:15PM +0530, Lokesh Janghel wrote: >>> Hi Mateuszb, >>> >>> I tested with your proposition patch and it is working right. >>> I also added the patch with test case. >>>

Re: [Patch] Bug 88521 - gcc 9.0 from r266355 miscompile x265 for mingw-w64 target

2018-12-28 Thread JonY
On 12/26/18 1:10 PM, Lokesh Janghel wrote: > Hi, > > Here is the patch for the issue. > Please let me know your thoughts. > > On Fri, Dec 21, 2018 at 3:16 PM JonY <10wa...@gmail.com> wrote: >> >> On 12/21/18 9:08 AM, Uros Bizjak wrote: >>> On Thu,

Re: [Mingw-w64-public] Fwd: [patch] Reimplement GNU threads library on native Windows

2019-07-29 Thread JonY
On 7/3/19 12:55 PM, Liu Hao wrote: > 在 2019/7/2 下午8:27, Jonathan Wakely 写道: >> >> What do you mean by "unclosed thread"? If I read it correctly, the MSDN >> page >> refers to closing a handle (which makes sense), not closing a thread. >> > > Yes, it meant a thread which has terminated but not dele

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-12 Thread JonY
On 10/12/2018 05:56 AM, Alexandre Oliva wrote: > On Oct 11, 2018, JonY <10wa...@gmail.com> wrote: > >> On 10/11/2018 02:57 AM, NightStrike wrote: >>> >>> Except that options typically don't get removed, just deprecated. It >>> seems cleaner to

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-11-01 Thread JonY
On 10/31/2018 10:39 PM, Alexandre Oliva wrote: > On Oct 12, 2018, Alexandre Oliva wrote: > >> * config/i386/mingw32.h (LINK_SPEC_LARGE_ADDR_AWARE): Define, >> based on MINGW_DEFAULT_LARGE_ADDR_AWARE. >> (LINK_SPEC): Insert it. >> * config/i386/mingw-264.h: Likewise. >

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-11-07 Thread JonY
On 11/07/2018 08:34 AM, Alexandre Oliva wrote: > On Nov 1, 2018, JonY wrote: > >> Looks like it causes an error on 64bit: >> /usr/libexec/gcc/x86_64-w64-mingw32/ld: unrecognized option >> '--large-address-aware' > > What does? The patch I suggeste

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-11-08 Thread JonY
On 11/08/2018 09:45 AM, Alexandre Oliva wrote: > On Nov 7, 2018, JonY <10wa...@gmail.com> wrote: > >> On 11/07/2018 08:34 AM, Alexandre Oliva wrote: >>> On Nov 1, 2018, JonY wrote: >>> >>>> Looks like it causes an error on 64bit: >>>>

Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-11-09 Thread JonY
On 11/09/2018 10:48 AM, Alexandre Oliva wrote: > On Nov 8, 2018, JonY <10wa...@gmail.com> wrote: > >> No, no. By quick I just mean using -Wl,--large-address-aware on an >> existing gcc install, nothing complex. Sorry about not making it clear. > > Ah, good! >

Re: [PATCH] [MinGW] Set __USE_MINGW_ACCESS for C++ as well

2019-03-18 Thread JonY
On 3/3/19 10:41 AM, Johannes Pfau wrote: > We set __USE_MINGW_ACCESS for windows hosts to use MinGWs wrapper > for the access function. The wrapper ensures that access behaves > in the expected way (e.g. for special files, such as nul). > However, we now compile most sources with the C++ compiler a

Re: [PATCH] [MinGW] Set __USE_MINGW_ACCESS for C++ as well

2019-03-19 Thread JonY
On 3/18/19 10:31 PM, JonY wrote: > On 3/3/19 10:41 AM, Johannes Pfau wrote: >> We set __USE_MINGW_ACCESS for windows hosts to use MinGWs wrapper >> for the access function. The wrapper ensures that access behaves >> in the expected way (e.g. for special files, such as nul

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-08-19 Thread JonY
On 5/26/2016 20:36, JonY wrote: > Text mode translation should not be done for Cygwin, especially since it does > not > support unicode setmode calls. This also fixes ada builds for Cygwin. > > OK for trunk? Ping? signature.asc Description: OpenPGP digital signature

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-08-19 Thread JonY
On 8/19/2016 20:49, Arnaud Charlet wrote: >>> Text mode translation should not be done for Cygwin, especially since it >>> does not >>> support unicode setmode calls. This also fixes ada builds for Cygwin. >>> >>> OK for trunk? >> >> Ping? > > Can you send the link to your original submission for

[Patch] Disable text mode translation in ada for Cygwin

2016-05-26 Thread JonY
Text mode translation should not be done for Cygwin, especially since it does not support unicode setmode calls. This also fixes ada builds for Cygwin. OK for trunk? gcc/ada/ChangeLog: * sysdep.c (__gnat_set_binary_mode, __gnat_set_text_mode, __gnat_set_mode): Disable text mode t

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-05-26 Thread JonY
On 5/26/2016 21:55, Arnaud Charlet wrote: >> Text mode translation should not be done for Cygwin, especially since it >> does not >> support unicode setmode calls. This also fixes ada builds for Cygwin. >> >> OK for trunk? > > OK, thanks. > Can someone please commit this? I don't have SVN write

Re: Revert gcc r227962

2016-05-29 Thread JonY
On 5/23/2016 16:56, JonY wrote: > On 5/20/2016 06:36, JonY wrote: >> On 5/20/2016 02:11, Jeff Law wrote: >>> So if we make this change (revert 227962), my understanding is that >>> cygwin bootstraps will fail because they won't find kernel32 and perhaps &

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-06-01 Thread JonY
On 5/27/2016 06:25, JonY wrote: > On 5/26/2016 21:55, Arnaud Charlet wrote: >>> Text mode translation should not be done for Cygwin, especially since it >>> does not >>> support unicode setmode calls. This also fixes ada builds for Cygwin. >>> >>

Re: Deprecate DBX/stabs?

2017-07-21 Thread JonY
On 07/21/2017 01:07 PM, Nathan Sidwell wrote: > [darwin, cygwin, rx maintainers, you might have an opinion] > Let's at least deprecate it. I attach a patch to do so. With the > patch, you'll get a note about dbx being deprecated whenever you use > stabs debugging on a system that prefers stabs (t

[PATCH][cygming] Share mingw fset-stack-executable with cygwin

2017-08-07 Thread JonY
This patch is in use by Cygwin for years, upstream to GCC. Committed to trunk. * gcc/config/i386/mingw.opt (fset-stack-executable): Removed. * gcc/config/i386/cygming.opt (fset-stack-executable): Moved from mingw.opt. * gcc/config/i386/cygwin.h: Define CHECK_EXECUT

[PATCH] Enable libitm DLL build on Cygwin/Mingw

2017-08-09 Thread JonY
Fixes libtool calls in libitm. Patch OK for trunk? 2017-08-09 Jonathan Yong <10wa...@gmail.com> * configure.ac: Check libtool flags. * Makefile.am: Use lt_host_flags. * configure: Regenerated. * Makefile.in: Regenerated. * testsuite/Makefile.in: Regenerat

Re: [PATCH] [i386] PR 81850 Don't ignore -mabi=sysv on Cygwin/MinGW

2017-08-22 Thread JonY
On 08/22/2017 06:32 AM, Uros Bizjak wrote: > On Tue, Aug 22, 2017 at 4:10 AM, Daniel Santos > wrote: > >> This is a problem that occured because of this code in >> ix86_option_override_internal: >> >> if (!opts_set->x_ix86_abi) >> opts->x_ix86_abi = DEFAULT_ABI; >> >> I tested this along w

Re: [PATCH] Enable libitm DLL build on Cygwin/Mingw

2017-08-25 Thread JonY
On 08/09/2017 01:02 PM, JonY wrote: > Fixes libtool calls in libitm. Patch OK for trunk? > > 2017-08-09 Jonathan Yong <10wa...@gmail.com> > > * configure.ac: Check libtool flags. > * Makefile.am: Use lt_host_flags. > * configure: Regener

Re: Heads-Up: early LTO debug to land, breaking Mach-O / [X]COFF

2017-05-12 Thread JonY
On 05/12/2017 09:24 AM, Richard Biener wrote: > > This is a heads-up that I am in the process of implementing the last > of Jasons review comments on the dwarf2out parts of early LTO debug > support. I hope to post final patches early next week after thoroughly > re-testing everything. > > Note

Re: [PATCH, GCC/x86 mingw32] Add configure option to force wildcard behavior on Windows

2017-01-18 Thread JonY
On 01/18/2017 09:48 AM, Thomas Preudhomme wrote: > By default, wildcard support on Windows for programs compiled with mingw > depends on how the mingw runtime was configured. This means if one wants > to build GCC for Windows with a consistent behavior with Wildcard > (enabled or disabled) the ming

Re: [wwwdocs] gcc-8/porting_to.html

2017-03-22 Thread JonY
On 03/22/2017 05:39 PM, Thomas Preudhomme wrote: > Hi JonY, > > Sorry for the delay. Please find updated patch as per Gerald and you > suggestion. > > Is this ok for wwwdocs once [1] is committed in GCC 8 cycle? > > [1] https://gcc.gnu.org/ml/gcc-patches/2017-02/msg0115

Re: [RFC] [PATCH v3 0/8] [i386] Use out-of-line stubs for ms_abi pro/epilogues

2017-03-30 Thread JonY
On 03/30/2017 05:50 PM, Daniel Santos wrote: > I have finally completed all tests for Cygwin and MinGW both 32- & > 64-bit with no additional test failures. There are still 567 tests > failing both pre- and post-patch with error "error while loading shared > libraries: cyggfortran-4.dll: cannot op

[PATCH] Suppress ssp.c unused variable warning on mingw

2017-03-31 Thread JonY
Hi, This suppresses an unused variable warning for mingw*, patch OK? Index: libssp/ssp.c === --- libssp/ssp.c(revision 246630) +++ libssp/ssp.c(working copy) @@ -72,7 +72,6 @@ __guard_setup (void) { unsigned char

Re: [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

2017-05-02 Thread JonY
On 05/01/2017 11:31 AM, Uros Bizjak wrote: > On Thu, Apr 27, 2017 at 10:04 AM, Daniel Santos > wrote: >> All of patches are concerned with 64-bit Microsoft ABI functions that call >> System V ABI function which clobbers RSI, RDI and XMM6-15 and are aimed at >> improving performance and .text size

Re: [PATCH] Fix cross compiling to x86_64-w64-mingw32

2017-05-02 Thread JonY
On 05/02/2017 12:11 PM, Hugo Beauzée-Luyssen wrote: > This patch fixes cross compiling to x86_64-w64-mingw32 > See https://github.com/Alexpux/MINGW-packages/issues/1580 and > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69506 > > My apologies if I missed something in the contributing/sending a pat

Re: [wwwdocs] gcc-8/porting_to.html

2017-05-04 Thread JonY
On 03/23/2017 10:47 AM, Thomas Preudhomme wrote: > Ack. Please find updated patch as per suggestions. > > Best regards, > > Thomas > I've applied the changes to GCC 8 trunk as r247588. signature.asc Description: OpenPGP digital signature

Re: [PATCH][mingw] Enable colorized diagnostics

2017-09-27 Thread JonY
On 09/27/2017 08:54 PM, Liu Hao wrote: > On 2017/9/28 4:09, Joseph Myers wrote: >> On Thu, 28 Sep 2017, Liu Hao wrote: >> >>> Colorized diagnostics used to be disabled for MinGW targets (on which >>> the macro `_WIN32` is defined), and this patch enables it. >> >> I'd hope this is all to do with Mi

Re: [PATCH][mingw] Enable colorized diagnostics

2017-10-09 Thread JonY
On 10/08/2017 11:39 AM, Liu Hao wrote: > On 2017/9/28 4:09, Joseph Myers wrote: >> On Thu, 28 Sep 2017, Liu Hao wrote: >> >>> Colorized diagnostics used to be disabled for MinGW targets (on which >>> the macro `_WIN32` is defined), and this patch enables it. >> >> I'd hope this is all to do with Mi

Re: [PATCH][mingw] Enable colorized diagnostics

2017-10-11 Thread JonY
On 10/09/2017 01:07 PM, Liu Hao wrote: > On 2017/10/9 19:01, JonY wrote: >> On 10/08/2017 11:39 AM, Liu Hao wrote: >> >> I'm not sure if it should be enabled by default due to the interleaving >> problem, but seeing as the user has to go out to set GCC_COLORS to u

Re: [PATCH, GCC/x86 mingw32] Add configure option to force wildcard behavior on Windows

2017-02-07 Thread JonY
On 01/26/2017 01:04 PM, Thomas Preudhomme wrote: > Hi JonY, > > On 19/01/17 01:37, JonY wrote: >> On 01/18/2017 09:48 AM, Thomas Preudhomme wrote: >>> By default, wildcard support on Windows for programs compiled with mingw >>> depends on how the mingw runtime

Re: [RFC] [PATCH v3 0/8] [i386] Use out-of-line stubs for ms_abi pro/epilogues

2017-02-10 Thread JonY
On 02/10/2017 10:32 AM, Uros Bizjak wrote: > On Tue, Feb 7, 2017 at 7:38 PM, Daniel Santos wrote: > >> Uros or Jan, >> Please take this as a ping, as I never bothered pinging after submitting v2 >> since I found a few more issues with it. :) Although I realize this would be >> a GCC 8 stage 1 ite

Re: [RFC] [PATCH v3 0/8] [i386] Use out-of-line stubs for ms_abi pro/epilogues

2017-02-10 Thread JonY
On 02/10/2017 05:23 PM, Daniel Santos wrote: > On 02/10/2017 05:34 AM, JonY wrote: >> Hi, >> mingw-w64 itself does not use any ms_abi/sysv_abi marked functions >> internally, so it should be unaffected. I don't think Cygwin uses any >> either, but I need to double

Re: [PATCH, GCC/x86 mingw32] Add configure option to force wildcard behavior on Windows

2017-02-14 Thread JonY
On 02/14/2017 09:32 AM, Thomas Preudhomme wrote: >> >> Looks good, be sure to emphasize this option affects mingw hosted GCC >> only, not the compiler output. > > I think that should be pretty clear in the latest version of the patch, > doc/install.texi contains: > > "Note that this option only a

Re: [PATCH, GCC/x86 mingw32] Add configure option to force wildcard behavior on Windows

2017-02-17 Thread JonY
On 02/14/2017 10:42 AM, JonY wrote: > On 02/14/2017 09:32 AM, Thomas Preudhomme wrote: >>> >>> Looks good, be sure to emphasize this option affects mingw hosted GCC >>> only, not the compiler output. >> >> I think that should be pretty clear in the latest

Re: [PATCH, GCC/x86 mingw32] Add configure option to force wildcard behavior on Windows

2017-02-17 Thread JonY
On 02/17/2017 11:31 AM, Thomas Preudhomme wrote: > Here you are: > > 2017-01-24 Thomas Preud'homme > > * configure.ac (--enable-mingw-wildcard): Add new configurable > feature. > * configure: Regenerate. > * config.in: Regenerate. > * config/i386/driver-mingw32.

[PATCH] Suppress compiler warning in libgcc/unwind-seh.c

2017-03-01 Thread JonY
Patch OK? ChangeLog: * unwind-seh.c: Suppress warnings for RtlUnwindEx() calls. Index: libgcc/unwind-seh.c === --- libgcc/unwind-seh.c (revision 245806) +++ libgcc/unwind-seh.c (working copy) @@ -221,7 +221,7 @@ test is tha

Re: [PATCH] Suppress compiler warning in libgcc/unwind-seh.c

2017-03-02 Thread JonY
On 03/01/2017 11:36 AM, JonY wrote: > Patch OK? > > ChangeLog: > * unwind-seh.c: Suppress warnings for RtlUnwindEx() calls. > Applied, tested with x86_64-w64-mingw32 to trunk. signature.asc Description: OpenPGP digital signature

[PATCH] Fix libgfortran build for mingw*

2017-03-02 Thread JonY
Applied to trunk, tested with x86_64-w64-mingw32. Changelog: 2017-03-02 Jonathan Yong <10wa...@gmail.com> * config/i386/gthr-win32.h: Define NOGDI before windows.h include to prevent w32api CC_NONE macro clash with libgfortran. --- trunk/libgcc/config/i386/gthr

Re: [wwwdocs] gcc-8/porting_to.html

2017-03-09 Thread JonY
On 03/09/2017 11:22 AM, Thomas Preudhomme wrote: > Hi Jakub, > > I forgot to mention "ok for stage1?", my bad. JonY said he was queuing > the corresponding changes in GCC for stage1 so I was just trying to have > a complete story. I do not expect to commit this before th

Re: [wwwdocs] gcc-8/porting_to.html

2017-03-12 Thread JonY
On 03/12/2017 02:07 PM, Gerald Pfeifer wrote: > On Thu, 9 Mar 2017, Thomas Preudhomme wrote: >> JonY: what about the attached patch to document the change of behavior >> of GCC on Windows depending on the configure option used? > > +MinGW issues > + > +GCC on M

Re: [PATCH] Suppress compiler warning in libgcc/unwind-seh.c

2017-03-15 Thread JonY
On 03/15/2017 04:12 AM, Jeff Law wrote: > On 03/01/2017 04:36 AM, JonY wrote: >> Patch OK? >> >> ChangeLog: >> * unwind-seh.c: Suppress warnings for RtlUnwindEx() calls. > You know this stuff better than anyone else working with GCC. If you > think this is

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-07-01 Thread JonY
On 6/1/2016 18:27, JonY wrote: > On 5/27/2016 06:25, JonY wrote: >> On 5/26/2016 21:55, Arnaud Charlet wrote: >>>> Text mode translation should not be done for Cygwin, especially since it >>>> does not >>>> support unicode setmode calls. This also

Re: [Patch] Disable text mode translation in ada for Cygwin

2016-07-01 Thread JonY
On 7/1/2016 20:00, Arnaud Charlet wrote: >> >> ping2? Is there a dedicated list for ADA patches? > > This list is for submitting patches, which you have done, it is not > really about pinging for commits, which should preferably be done by > the submitter, after proper testing. > > I do not have

[PATCH] Add missing OBJCOPY variable to Makefile.in

2016-07-03 Thread JonY
This patch allows OBJCOPY to be set by configure. It was missing in Makefile.in. Patch OK? From c7b2b18116cd0ba73ff585299b5190a05e267d3f Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Sat, 26 Mar 2016 12:32:18 +0100 Subject: [PATCH] Add missing OBJCOPY variable to Makefile.in Signed-off-b

Re: [PATCH] Add missing OBJCOPY variable to Makefile.in

2016-07-13 Thread JonY
On 7/14/2016 06:22, Jeff Law wrote: > On 07/03/2016 05:56 AM, JonY wrote: >> This patch allows OBJCOPY to be set by configure. It was missing in >> Makefile.in. >> >> Patch OK? > With a ChangeLog and verification that some host/target combination > still builds

Re: [PATCH] Add missing OBJCOPY variable to Makefile.in

2016-07-14 Thread JonY
On 7/14/2016 06:22, Jeff Law wrote: > On 07/03/2016 05:56 AM, JonY wrote: >> This patch allows OBJCOPY to be set by configure. It was missing in >> Makefile.in. >> >> Patch OK? > With a ChangeLog and verification that some host/target combination > s

Re: Revert gcc r227962

2016-05-19 Thread JonY
On 5/20/2016 02:11, Jeff Law wrote: > So if we make this change (revert 227962), my understanding is that > cygwin bootstraps will fail because they won't find kernel32 and perhaps > other libraries. > > Jeff > I'll need to double check with trunk but gcc-5.3.0 built OK without it. The other alt

Re: Revert gcc r227962

2016-05-23 Thread JonY
On 5/20/2016 06:36, JonY wrote: > On 5/20/2016 02:11, Jeff Law wrote: >> So if we make this change (revert 227962), my understanding is that >> cygwin bootstraps will fail because they won't find kernel32 and perhaps >> other libraries. >> >> Jeff >> &

Re: [PATCH][GCC] Fix native Windows x86 bootstrap failure with self test

2016-11-04 Thread JonY
On 11/4/2016 21:22, Tamar Christina wrote: > Hi all, > > The GCC self-test added in r237144 breaks the > native Windows x86 builds (e.g. mingw-w64). > This fixes (PR78196) by explicitly adding /dev/null as > the output file to the GCC self test. > > The test essentially does `-xc -S -c /dev/null

Minor Cygwin patches

2013-07-10 Thread JonY
Hi, Attached are some minor patches, comments? http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35536 --- origsrc/gcc-4.5.1/libjava/gnu/java/nio/channels/natFileChannelPosix.cc 2007-04-02 20:05:50.0 -0500 +++ src/gcc-4.5.1/libjava/gnu/java/nio/channels/natFileChannelPosix.cc 2010-10-27

Re: Minor Cygwin patches

2013-07-13 Thread JonY
On 7/10/2013 20:43, JonY wrote: > Hi, > > Attached are some minor patches, comments? > Kai, ping? signature.asc Description: OpenPGP digital signature

Re: Minor Cygwin patches

2013-07-14 Thread JonY
On 7/15/2013 06:02, Kai Tietz wrote: > 2013/7/13 JonY <10wa...@gmail.com>: >> On 7/10/2013 20:43, JonY wrote: >>> Hi, >>> >>> Attached are some minor patches, comments? >>> >> >> Kai, ping? > > > Please sent each patch as s

Re: Minor Cygwin patches

2013-07-14 Thread JonY
On 7/15/2013 06:02, Kai Tietz wrote: > 2013/7/13 JonY <10wa...@gmail.com>: >> On 7/10/2013 20:43, JonY wrote: >>> Hi, >>> >>> Attached are some minor patches, comments? >>> >> >> Kai, ping? > > > Please sent each patch as s

Re: Minor Cygwin patches

2013-07-14 Thread JonY
On 7/15/2013 06:02, Kai Tietz wrote: > 2013/7/13 JonY <10wa...@gmail.com>: >> On 7/10/2013 20:43, JonY wrote: >>> Hi, >>> >>> Attached are some minor patches, comments? >>> >> >> Kai, ping? > > > Please sent each patch as s

Re: Minor Cygwin patches

2013-07-23 Thread JonY
On 7/19/2013 15:21, Yaakov (Cygwin/X) wrote: > On 2013-07-14 20:15, JonY wrote: >> 2013-03-08 Yaakov Selkowitz >> >> * (gcc/testsuite/gcc.target/i386/pr25993.c): Skip unsupported test. > > This patch was Dave Korn's. > > OK. Changelog 2013-03

Re: Minor Cygwin patches

2013-07-23 Thread JonY
On 7/19/2013 15:23, Yaakov (Cygwin/X) wrote: > On 2013-07-14 20:15, JonY wrote: >> 2013-03-08 Dave Korn >> >> * (gcc/config.gcc): Include Cygwin specific file. >> * (gcc/config/i386/cygwin.h): Link shared libgcc by default. >> * (gcc/config/i3

[Patch] Don't test for pr53425 on mingw

2012-06-18 Thread JonY
Hi, I am told that this ABI test does not apply to mingw targets. OK to apply? Index: gcc/testsuite/gcc.target/i386/pr53425-1.c === --- gcc/testsuite/gcc.target/i386/pr53425-1.c (revision 188384) +++ gcc/testsuite/gcc.target/i386/pr

[PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
Hi, For correctness sake, determine version from BASE-VER file instead of calling compiler. This allows better toleration with bad Canadian cross setups where host gcc version is of different version from target. Patch OK? Index: gcc-trunk/libgcc/Makefile.in =

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/3/2012 22:37, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: >> >> For correctness sake, determine version from BASE-VER file instead of >> calling compiler. This allows better toleration with bad Canadian cross >> setups where host gcc vers

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/3/2012 23:35, Ian Lance Taylor wrote: > On Fri, Aug 3, 2012 at 8:02 AM, JonY wrote: >> On 8/3/2012 22:37, Ian Lance Taylor wrote: >>> On Fri, Aug 3, 2012 at 7:01 AM, JonY wrote: >>>> >>>> For correctness sake, determine version from BASE-VER f

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/4/2012 13:14, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: >>> >>> OK. Btw, I don't actually have write access, so please commit if the >>> ChangeLog i

Re: [PATCH] Fix libgcc version for Canadian cross compile

2012-08-03 Thread JonY
On 8/4/2012 13:24, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 10:14 PM, H.J. Lu wrote: >> On Fri, Aug 3, 2012 at 4:42 PM, Ian Lance Taylor wrote: >>> On Fri, Aug 3, 2012 at 9:04 AM, JonY wrote: >>>> >>>> OK. Btw, I don't actually have write access,

[patch] --enable-dynamic-string default for mingw-w64

2011-09-19 Thread JonY
Hi, Its been used in the automated toolchain builds for sometime, seems like a good idea to enable it by default. It can be easily changed to match for all mingw as well if needed. OK for trunk? Index: libstdc++-v3/configure.ac ===

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread JonY
On 9/20/2011 13:59, Kai Tietz wrote: > 2011/9/20 JonY: >> Hi, >> >> Its been used in the automated toolchain builds for sometime, seems like >> a good idea to enable it by default. It can be easily changed to match >> for all mingw as well if needed. >> >

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread JonY
On 9/20/2011 22:12, Kai Tietz wrote: >> I'm not really opposed to making this change for i*86-pc-mingw -- and >> now's the time to do it, as the recently released 4.6.1 mingw.org gcc >> broke the C++ abi anyway, thanks to thiscall. > > Here I am a bit curious? How is 4.6.1 affected by new thiscal

Re: [Mingw-users] [patch] --enable-dynamic-string default for mingw-w64

2011-09-23 Thread JonY
On 9/21/2011 17:08, xunxun wrote: > 于 2011/9/21 10:56, Cesar Strauss 写道: >> Please let me present an opposing view. >> >> On the one hand, according to comment 4 of [1], by using >> --enable-fully-dynamic-string, all other users will miss a very good >> optimization. On the other hand, these users

  1   2   >