ilinpv added a comment.

@sebpop could you ellaborate on __sync_* operations usage, are you getting 
issues with current Clang implementation? Do Clang need to keep supporting them 
and fix introducing new memory model? It seems we need compelling reasons to do 
that.

>> However if sync primitives are hardly used in the real world then perhaps it 
>> is about time to deprecate them with annoying warnings, and completely 
>> remove support next year. Does that sound reasonable?
>
> I don't know that they are "hardly used" in the real world -- there is 
> certainly legacy code using them, although I haven't attempted to quantify 
> the amount. Another interesting question would be: how much of that code is 
> actually correct, at all. I'd be rather surprised if someone could identify a 
> single piece of software which is actually correct when using the __sync_* 
> operations on GCC, but is incorrect on Clang.




Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129802/new/

https://reviews.llvm.org/D129802

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to