[PATCH] D25068: [coroutines] Switch to using std::experimental namespace per P0057R5

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 30 17:55:44 PDT 2016


EricWF marked an inline comment as done.
EricWF added inline comments.


> rsmith wrote in SemaDeclCXX.cpp:8157-8160
> If you're going to track the namespace on creation, you need to add serialization code for your `Sema::StdExperimentalNamespace` storage. But I'd suggest taking a different approach: just treat this as a cache, and do a lookup for `experimental` within namespace `std` if the namespace is requested and not in the cache.

Done. Could you please double check I got it correct?

https://reviews.llvm.org/D25068





More information about the cfe-commits mailing list