[PATCH] D85532: Correctly set CompilingPCH in PrecompilePreambleAction.
Adam Czachorowski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 10 07:43:22 PDT 2020
adamcz added inline comments.
================
Comment at: clang/unittests/Frontend/ASTUnitTest.cpp:146
+ auto File = AU->getFileManager().getFileRef("Textual.h", false, false);
+ assert(File);
+ // Verify that we do not crash here.
----------------
kadircet wrote:
> ASSERT_TRUE again
You can 't ASSERT_TRUE() on llvm::Expected, since operator bool() is not const. No idea why, but that's why it's assert()
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85532/new/
https://reviews.llvm.org/D85532
More information about the cfe-commits
mailing list