[PATCH] D83340: Prohibit use of _ExtInt in atomic intrinsic

2020-07-14 Thread Erich Keane via Phabricator via cfe-commits
erichkeane added a comment. In D83340#2149100 , @jtmott-intel wrote: > @jfb Ah, I feel I better understand your original question now. Thinking > through how we would want `__atomic`s to behave is great, and I'm genuinely > glad someone's championing it

[PATCH] D83340: Prohibit use of _ExtInt in atomic intrinsic

2020-07-14 Thread Ronan Keryell via Phabricator via cfe-commits
keryell added a comment. The idea of this `_ExtInt` is to have some extensions. Since it is an extension, why preventing its use? For example if I want my 18 bit FPGA BRAM to be accessed atomically? Or is there an assumption that atomic access can be enabled back with some other mode, such as SYC

[PATCH] D83340: Prohibit use of _ExtInt in atomic intrinsic

2020-07-14 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGd083adb068e7: Prohibit use of _ExtInt in atomic intrinsic (authored by jtmott-intel, committed by erichkeane). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM