[cfe-dev] FYI, Intel folks might be looking to add the __iso_volatile_Xxx family for MSVC STL <atomic> soon

Reid Kleckner via cfe-dev cfe-dev at lists.llvm.org
Thu Mar 28 14:18:17 PDT 2019


On Thu, Mar 28, 2019 at 2:14 PM Reid Kleckner <rnk at google.com> wrote:

> On Thu, Mar 28, 2019 at 1:46 PM JF Bastien <jfbastien at apple.com> wrote:
>
>> On Mar 28, 2019, at 12:02 PM, Billy O'Neal (VC LIBS) <bion at microsoft.com>
>> wrote:
>>
>> Update: VS2019 Update 2 is likely to do this; who do I need to prod such
>> that I get to use these unconditionally in our <atomic> ?🙂
>>
>>
>> You want the same feature in clang-cl, so I suggest talking to Reid,
>> sending a patch to clang, and having him review it ;-)
>>
>
> I think it's just a matter of moving code from BuiltinsAArch64.def
> <https://github.com/llvm/llvm-project/blob/master/clang/include/clang/Basic/BuiltinsAArch64.def#L82> (and
> ARM) to Builtins.def, and maybe some followon changes.
>
> The naming for these intrinsics is confusing. It reads as, "do an ISO C++
> standard volatile load", but it really means "do an atomic (not volatile)
> load", but I suppose the time to address that is long gone.
>

Huh, maybe I'm the one confused, because all they do is emit a volatile
load. What exactly do you want to use these for, then? I'm not sure we've
implemented them correctly.


> I went ahead and started a patch to make them available.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20190328/d7d40c39/attachment.html>


More information about the cfe-dev mailing list