tra added a comment. In D107492#2926871 <https://reviews.llvm.org/D107492#2926871>, @JonChesterfield wrote:
> Therefore I'd like to leave it as `__asm__ volatile`. Being the one who introduced inconsistent use of `__volatile__` and `volatile` in this header, I'm pretty sure that PTX's notion of volatile is not involved for inline asm of `trap` and `brkpt`. It should be fine changing them to `__volatile__` for the sake of uniformity. Up to you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107492/new/ https://reviews.llvm.org/D107492 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits