public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/48023] New: [trans-mem] no-builtin flag and TM builtin like memset
@ 2011-03-07 16:23 patrick.marlier at gmail dot com
  2011-11-21 14:05 ` [Bug c/48023] " torvald at gcc dot gnu.org
  0 siblings, 1 reply; 2+ messages in thread
From: patrick.marlier at gmail dot com @ 2011-03-07 16:23 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48023

           Summary: [trans-mem] no-builtin flag and TM builtin like memset
           Product: gcc
           Version: trans-mem
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: patrick.marlier@gmail.com
                CC: rth@gcc.gnu.org, aldyh@gcc.gnu.org


Created attachment 23574
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23574
testcase for memset.

If -fgnu-tm and -fno-builtin are used, code is transactified but then the
memset becomes unsafe. 
I think the behavior is correct but I just want to be sure. So don't hesitate
to mark it as invalid.

$ gcc -fno-builtin -fgnu-tm -S -O0 testcase.i
testcase.i:10:8: error: unsafe function call ‘memcpy’ within ‘transaction_safe’
function

$ gcc -fgnu-tm -S -O0 testcase.i

Patrick Marlier.


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

* [Bug c/48023] [trans-mem] no-builtin flag and TM builtin like memset
  2011-03-07 16:23 [Bug c/48023] New: [trans-mem] no-builtin flag and TM builtin like memset patrick.marlier at gmail dot com
@ 2011-11-21 14:05 ` torvald at gcc dot gnu.org
  0 siblings, 0 replies; 2+ messages in thread
From: torvald at gcc dot gnu.org @ 2011-11-21 14:05 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48023

torvald at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |torvald at gcc dot gnu.org
         Resolution|                            |FIXED

--- Comment #1 from torvald at gcc dot gnu.org 2011-11-21 13:27:11 UTC ---
This works for current trunk (both for memset and the memcpy in your test
case).


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

end of thread, other threads:[~2011-11-21 13:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-07 16:23 [Bug c/48023] New: [trans-mem] no-builtin flag and TM builtin like memset patrick.marlier at gmail dot com
2011-11-21 14:05 ` [Bug c/48023] " torvald 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).