harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Varlamov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-5727) [port][atomic] implement port_atomic_casptr for Windows
Date Fri, 18 Apr 2008 05:09:22 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12590288#action_12590288
] 

Alexey Varlamov commented on HARMONY-5727:
------------------------------------------

Alexey,
MSDN is quite distinct that arguments to InterlockedCompareExchangePointer must be aligned
[1]:
"On a 64-bit system, the parameters are 64 bits and must be aligned on 64-bit boundaries;
otherwise, the function will behave unpredictably. On a 32-bit system, the parameters are
32 bits and must be aligned on 32-bit boundaries."

Yet the header already defines other port_atomic_cas* functions referring to VS intrinsics
on Win64.
I believe some more research is needed to set these right.

[1] http://msdn2.microsoft.com/en-us/library/ms683568(VS.85).aspx

> [port][atomic] implement port_atomic_casptr for Windows
> -------------------------------------------------------
>
>                 Key: HARMONY-5727
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5727
>             Project: Harmony
>          Issue Type: Sub-task
>          Components: DRLVM
>            Reporter: Alexei Fedotov
>         Attachments: atomic.patch
>
>
> apr_atomic_casptr requires APR to be initialized, does not inline and use modern processor
instructions. The patch extends port functionality to supported platforms. One day it would
be better committed into APR.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message