aaron.ballman added a comment. In D146187#4201223 <https://reviews.llvm.org/D146187#4201223>, @ChuanqiXu wrote:
>> but we don't define __cpp_impl_coroutine: >> http://eel.is/c++draft/tab:cpp.predefined.ft > > We defined `__cpp_impl_coroutine`. And `__cpp_coroutines` is for > Coroutines TS. I forgot to remove this too. I'll handle it in a separate > patch. Thanks for handling the TS bits, but what am I misunderstanding here: https://godbolt.org/z/KdrM713r5 ? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146187/new/ https://reviews.llvm.org/D146187 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
