Grokbase › Groups › Go › golang-dev › February 2015
FAQ
Hi,

I realize it's probably not the time to fix this now, but I was reading
the linker source and found a comment which lead me to check and find
that it seems the cfunc field on LSym is never set in current code,
which also means that there are no references to morestackc or the
stackguard1 field on g. Have I missed something, or can all this code
be ripped out? (After the c2go stuff has landed, presumably).

Cheers,
mwh

--
You received this message because you are subscribed to the Google Groups "golang-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to golang-dev+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Search Discussions

  • Minux at Feb 12, 2015 at 11:29 pm ⇧
    FWIW, I tried to remove stackguard1 in CL 2144, but Russ reverted my CL
    later.
    On Feb 12, 2015 6:23 PM, "Michael Hudson-Doyle" wrote:
    I realize it's probably not the time to fix this now, but I was reading
    the linker source and found a comment which lead me to check and find
    that it seems the cfunc field on LSym is never set in current code,
    which also means that there are no references to morestackc or the
    stackguard1 field on g. Have I missed something, or can all this code
    be ripped out? (After the c2go stuff has landed, presumably).
    --
    You received this message because you are subscribed to the Google Groups "golang-dev" group.
    To unsubscribe from this group and stop receiving emails from it, send an email to golang-dev+unsubscribe@googlegroups.com.
    For more options, visit https://groups.google.com/d/optout.
  • Russ Cox at Feb 13, 2015 at 10:05 pm ⇧

    On Thu, Feb 12, 2015 at 6:22 PM, Michael Hudson-Doyle wrote:

    Hi,

    I realize it's probably not the time to fix this now, but I was reading
    the linker source and found a comment which lead me to check and find
    that it seems the cfunc field on LSym is never set in current code,
    which also means that there are no references to morestackc or the
    stackguard1 field on g. Have I missed something, or can all this code
    be ripped out? (After the c2go stuff has landed, presumably).
    It's not used today but I intend to start using it, to mark certain runtime
    functions as "must run on g0 stack".

    Russ

    --
    You received this message because you are subscribed to the Google Groups "golang-dev" group.
    To unsubscribe from this group and stop receiving emails from it, send an email to golang-dev+unsubscribe@googlegroups.com.
    For more options, visit https://groups.google.com/d/optout.
  • Michael Hudson-Doyle at Feb 14, 2015 at 1:41 am ⇧

    Russ Cox writes:

    On Thu, Feb 12, 2015 at 6:22 PM, Michael Hudson-Doyle <
    michael.hudson@canonical.com> wrote:
    Hi,

    I realize it's probably not the time to fix this now, but I was reading
    the linker source and found a comment which lead me to check and find
    that it seems the cfunc field on LSym is never set in current code,
    which also means that there are no references to morestackc or the
    stackguard1 field on g. Have I missed something, or can all this code
    be ripped out? (After the c2go stuff has landed, presumably).
    It's not used today but I intend to start using it, to mark certain runtime
    functions as "must run on g0 stack".
    Ah, cool, so s/cfunc/g0func/ or similar. Thanks for the clarification.

    Cheers,
    mwh

    --
    You received this message because you are subscribed to the Google Groups "golang-dev" group.
    To unsubscribe from this group and stop receiving emails from it, send an email to golang-dev+unsubscribe@googlegroups.com.
    For more options, visit https://groups.google.com/d/optout.

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedFeb 12, '15 at 11:23p
activeFeb 14, '15 at 1:41a
posts4
users3
websitegolang.org

3 users in discussion

Michael Hudson-Doyle: 2 posts Russ Cox: 1 post Minux: 1 post

Content

People

Support

Translate

site design / logo © 2022 Grokbase