[PATCH] D28835: [coroutines] NFC: Refactor Sema::CoroutineBodyStmt construction.

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 10 14:28:45 PST 2017


EricWF added a comment.

This LGTM after applying the fixes.



================
Comment at: lib/Sema/SemaCoroutine.cpp:719
+
+  if (!PromiseRecordDecl)
+    return true;
----------------
I figured out what's going on. `PromiseRecordDecl` doesn't get initialized to null when `IsPromiseDependentType` is false. Initializing `PromiseRecordDecl` fixes the problem.


https://reviews.llvm.org/D28835





More information about the cfe-commits mailing list