ChuanqiXu added a comment.

> Should we also be updating InitPreprocessor.cpp at the same time, for 
> non-Windows targets?

I didn't address this since we didn't do this before. (Defining feature macro 
according to the targets).

Also I feel it may be bad for windows users. Since currently the coroutines on 
windows is not completely broken. It is still workable in some (or a lot?) 
situations. It is just out of maintenance.


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

Reply via email to