> I understand that the name "coroutines-ts" isn't meaningful these > days, but it also sounds like this commit does more than remove the > flag, it caps useful functionality. How are users supposed to use > c++17 with coroutines now? It's very common in our codebase and we > have users relying on it.
Looks like I misread your patch, sorry! If my understanding now is right, `-std=c++17 -fcoroutines` should work, right? We should probably add an extra test in `clang/test/Driver/coroutines.cpp` that just test that (we currently just test when it's not). Cheers, -- Bruno Cardoso Lopes http://www.brunocardoso.cc _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits