[PATCH] D50672: [ASTImporter] Change the return result of Decl import to Optional

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 14 05:37:29 PDT 2018


martong added a comment.

> Do you think we can hold back this patch for a few days until Balazs prepares the Expected<T> based version? Then I think we could compare the patches and we could merge the best from the two of them.

I mean, once Balazs is also ready, then we can create a combined patch which holds the good things from both of yours and his patches.


Repository:
  rC Clang

https://reviews.llvm.org/D50672





More information about the cfe-commits mailing list