https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #28 from LIU Hao ---
(In reply to Andrew Pinski from comment #27)
> %zu should be added to ms_printf too.
MSVCRT.DLL from Windows 7 does not support the `z` modifier.
It seems supported on Windows 10; however people really should
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #27 from Andrew Pinski ---
%zu should be added to ms_printf too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #26 from LIU Hao ---
(In reply to Martin Storsjö from comment #25)
> But since the change in c51f1e7427e6a5ae2a6d82b5a790df77a3adc99a (released
> in GCC 12 already), we probably don't need this any longer. So I think it
> might be
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #25 from Martin Storsjö ---
(In reply to Andrew Pinski from comment #23)
> Note since MSVC 2015 runtime, printf has support %ll so ms_printf should be
> fixed to incldue that.
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #24 from LIU Hao ---
(In reply to Andrew Pinski from comment #23)
> Note since MSVC 2015 runtime, printf has support %ll so ms_printf should be
> fixed to incldue that.
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #23 from Andrew Pinski ---
Note since MSVC 2015 runtime, printf has support %ll so ms_printf should be
fixed to incldue that.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #22 from Tomas Kalibera ---
(In reply to CVS Commits from comment #21)
> The master branch has been updated by Jonathan Yong :
>
> https://gcc.gnu.org/g:966f3c134bb4802ac7ba0517de4e8e3f6384cfa3
>
> commit
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #21 from CVS Commits ---
The master branch has been updated by Jonathan Yong :
https://gcc.gnu.org/g:966f3c134bb4802ac7ba0517de4e8e3f6384cfa3
commit r14-3334-g966f3c134bb4802ac7ba0517de4e8e3f6384cfa3
Author: Tomas Kalibera
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #20 from Tomas Kalibera ---
(In reply to Julian Waters from comment #19)
> (In reply to Tomas Kalibera from comment #17)
> > (In reply to Tomas Kalibera from comment #16)
> > > (In reply to Julian Waters from comment #15)
> > > > It
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #19 from Julian Waters ---
(In reply to Tomas Kalibera from comment #17)
> (In reply to Tomas Kalibera from comment #16)
> > (In reply to Julian Waters from comment #15)
> > > It seems like the patch also doesn't fix the strftime
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #18 from Julian Waters ---
That's great news! With regards to this patch, I'm fairly certain you have to
proactively send it to gcc-patches mailing list for it to be merged; No gcc
committer has the time to look for this issue and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #17 from Tomas Kalibera ---
(In reply to Tomas Kalibera from comment #16)
> (In reply to Julian Waters from comment #15)
> > It seems like the patch also doesn't fix the strftime case too, strangely
> > enough. gcc with that patch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #16 from Tomas Kalibera ---
(In reply to Julian Waters from comment #15)
> It seems like the patch also doesn't fix the strftime case too, strangely
> enough. gcc with that patch applied still causes a compilation failure in
> the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
Julian Waters changed:
What|Removed |Added
CC||tanksherman27 at gmail dot com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
Alvin Wong changed:
What|Removed |Added
CC||alvinhochun at gmail dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #13 from Tomas Kalibera ---
Another instance of this problem is %z (to print size_t). It is supported by
UCRT, but the GCC's Microsoft format warns, because it wasn't supported with
MSVCRT (seen with GCC 12.2).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
Tomas Kalibera changed:
What|Removed |Added
Attachment #52007|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #11 from Tomas Kalibera ---
(In reply to LIU Hao from comment #10)
> (In reply to Tomas Kalibera from comment #7)
> > I sent an updated version for the trunk, 12, 11 and 10 to the gcc-patches
> > mailing list in May:
> >
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #10 from LIU Hao ---
(In reply to Tomas Kalibera from comment #7)
> I sent an updated version for the trunk, 12, 11 and 10 to the gcc-patches
> mailing list in May:
>
> https://gcc.gnu.org/pipermail/gcc-patches/2022-May/594960.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #9 from Tomas Kalibera ---
(In reply to Martin Storsjö from comment #8)
> (In reply to Tomas Kalibera from comment #7)
> > I sent an updated version for the trunk, 12, 11 and 10 to the gcc-patches
> > mailing list in May:
> >
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #8 from Martin Storsjö ---
(In reply to Tomas Kalibera from comment #7)
> I sent an updated version for the trunk, 12, 11 and 10 to the gcc-patches
> mailing list in May:
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #7 from Tomas Kalibera ---
I sent an updated version for the trunk, 12, 11 and 10 to the gcc-patches
mailing list in May:
https://gcc.gnu.org/pipermail/gcc-patches/2022-May/594960.html
The patches still apply to current 10,11,12
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
LIU Hao changed:
What|Removed |Added
CC||lh_mouse at 126 dot com
--- Comment #6 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #5 from Tomas Kalibera ---
(In reply to jos...@codesourcery.com from comment #1)
> See bug 92292. The extra attribute isn't ignored, it simply means the
> function has multiple format attributes, which is valid, but should
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #4 from Tomas Kalibera ---
Created attachment 52008
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52008=edit
Example from comment 2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #3 from Tomas Kalibera ---
Created attachment 52007
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52007=edit
Patch to ignore first format for builtins that have more than one
This patch for 10.3.0 instructs gcc to ignore the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
Tomas Kalibera changed:
What|Removed |Added
CC||tomas.kalibera at gmail dot com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95130
--- Comment #1 from joseph at codesourcery dot com ---
See bug 92292. The extra attribute isn't ignored, it simply means the
function has multiple format attributes, which is valid, but should
probably be special-cased for this case of
28 matches
Mail list logo
