public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem
@ 2015-07-15 18:24 dcb314 at hotmail dot com
2015-09-24 8:53 ` [Bug other/66887] " dcb314 at hotmail dot com
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: dcb314 at hotmail dot com @ 2015-07-15 18:24 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66887
Bug ID: 66887
Summary: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance
problem
Product: gcc
Version: 6.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: other
Assignee: unassigned at gcc dot gnu.org
Reporter: dcb314 at hotmail dot com
Target Milestone: ---
[trunk/libmpx/mpxrt/mpxrt.c:159]: (performance) Buffer 'buffer' is being
written before its old content has been used.
Source code is
memset (buffer, 0, sizeof (buffer));
memcpy (buffer,
(uint8_t *)uctxt->uc_mcontext.fpregs + XSAVE_OFFSET_IN_FPMEM,
sizeof (struct xsave_struct));
Maybe something like
memcpy (buffer,
(uint8_t *)uctxt->uc_mcontext.fpregs + XSAVE_OFFSET_IN_FPMEM,
sizeof (struct xsave_struct));
memset (&buffer[sizeof (struct xsave_struct))], 0, sizeof (buffer) - sizeof
(struct xsave_struct)) );
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug other/66887] trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem
2015-07-15 18:24 [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem dcb314 at hotmail dot com
@ 2015-09-24 8:53 ` dcb314 at hotmail dot com
2015-09-24 10:02 ` ienkovich at gcc dot gnu.org
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: dcb314 at hotmail dot com @ 2015-09-24 8:53 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66887
--- Comment #1 from David Binderman <dcb314 at hotmail dot com> ---
sizeof (struct xsave_struct) is about 900 bytes, so there is plenty of time
to be saved.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug other/66887] trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem
2015-07-15 18:24 [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem dcb314 at hotmail dot com
2015-09-24 8:53 ` [Bug other/66887] " dcb314 at hotmail dot com
@ 2015-09-24 10:02 ` ienkovich at gcc dot gnu.org
2015-10-15 9:27 ` ienkovich at gcc dot gnu.org
2015-10-15 9:30 ` ienkovich at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2015-09-24 10:02 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66887
Ilya Enkovich <ienkovich at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2015-09-24
CC| |ienkovich at gcc dot gnu.org
Assignee|unassigned at gcc dot gnu.org |ienkovich at gcc dot gnu.org
Ever confirmed|0 |1
--- Comment #2 from Ilya Enkovich <ienkovich at gcc dot gnu.org> ---
(In reply to David Binderman from comment #1)
> sizeof (struct xsave_struct) is about 900 bytes, so there is plenty of time
> to be saved.
Thanks for pointing at this. Actually I don't see my we have both memset and
memcpy here. Structure is copied just to return its field. I'll fix it.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug other/66887] trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem
2015-07-15 18:24 [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem dcb314 at hotmail dot com
2015-09-24 8:53 ` [Bug other/66887] " dcb314 at hotmail dot com
2015-09-24 10:02 ` ienkovich at gcc dot gnu.org
@ 2015-10-15 9:27 ` ienkovich at gcc dot gnu.org
2015-10-15 9:30 ` ienkovich at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2015-10-15 9:27 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66887
--- Comment #3 from Ilya Enkovich <ienkovich at gcc dot gnu.org> ---
Author: ienkovich
Date: Thu Oct 15 09:26:39 2015
New Revision: 228838
URL: https://gcc.gnu.org/viewcvs?rev=228838&root=gcc&view=rev
Log:
libmpx/
PR other/66887
* mpxrt/mpxrt.c (read_mpx_status_sig): Remove useless code.
Modified:
trunk/libmpx/ChangeLog
trunk/libmpx/mpxrt/mpxrt.c
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug other/66887] trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem
2015-07-15 18:24 [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem dcb314 at hotmail dot com
` (2 preceding siblings ...)
2015-10-15 9:27 ` ienkovich at gcc dot gnu.org
@ 2015-10-15 9:30 ` ienkovich at gcc dot gnu.org
3 siblings, 0 replies; 5+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2015-10-15 9:30 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66887
Ilya Enkovich <ienkovich at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #4 from Ilya Enkovich <ienkovich at gcc dot gnu.org> ---
Fixed
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-10-15 9:30 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-15 18:24 [Bug other/66887] New: trunk/libmpx/mpxrt/mpxrt.c:158: possible performance problem dcb314 at hotmail dot com
2015-09-24 8:53 ` [Bug other/66887] " dcb314 at hotmail dot com
2015-09-24 10:02 ` ienkovich at gcc dot gnu.org
2015-10-15 9:27 ` ienkovich at gcc dot gnu.org
2015-10-15 9:30 ` ienkovich at gcc dot gnu.org
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).