[PATCH] D37496: Fix validation of the -mthread-model flag in the Clang driver

Ian Tessier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 6 09:55:15 PDT 2017


itessier added a comment.

In https://reviews.llvm.org/D37496#861746, @compnerd wrote:

> The change looks good.  Can you please add some test cases for this?  Or do existing test cases cover this already?


Should have added this to the description, but yes there are existing tests that cover this flag (test/Driver/thread-model.c). https://reviews.llvm.org/D37493 removes the only overridden thread model methods so we can't test the fix in this CL.


https://reviews.llvm.org/D37496





More information about the cfe-commits mailing list