[PATCH] D125862: [clang][driver] Add gcc-toolset/devtoolset 12 to prefixes
Timm Bäder via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 2 00:35:05 PDT 2022
tbaeder marked 2 inline comments as done.
tbaeder added inline comments.
================
Comment at: clang/unittests/Driver/ToolChainTest.cpp:623
+ IntrusiveRefCntPtr<DiagnosticIDs> DiagID(new DiagnosticIDs());
+ struct TestDiagnosticConsumer : public DiagnosticConsumer {};
+
----------------
MaskRay wrote:
> delete `public`.
>
> Can you avoid using a derived DiagnosticConsumer?
I can use the `SimpleDiagnosticConsumer` class created earlier in the file.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125862/new/
https://reviews.llvm.org/D125862
More information about the cfe-commits
mailing list