mibintc added a comment.

In D59307#1427663 <https://reviews.llvm.org/D59307#1427663>, @jfb wrote:

> Thinking some more, this is really a silly gotcha in code. We should probably 
> follow what we're about to do with wg21.link/P1152 
> <https://reviews.llvm.org/P1152> and ban chaining. Statement expressions 
> should therefore not be allowed to return `_Atomic`, `std::atomic`, nor 
> `volatile`.


If the atomic load which is the value of the StmtExpr drops the atomic 
attribute then this would trivially be true, and there would be no need to add 
a restriction. According to what Clark says, once the value is loaded it's just 
data.


Repository:
  rC Clang

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

https://reviews.llvm.org/D59307



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

Reply via email to