[PATCH] D67364: [Driver] Handle default case in refactored addOpenMPRuntime
This revision was automatically updated to reflect the committed changes. Closed by commit rL371444: [Driver] Handle default case in refactored addOpenMPRuntime (authored by pirama, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D67364?vs=219413=219415#toc Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67364/new/ https://reviews.llvm.org/D67364 Files: cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp Index: cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp === --- cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp +++ cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + case Driver::OMPRT_Unknown: +break; } if (ForceStaticHostRuntime) Index: cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp === --- cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp +++ cfe/trunk/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + case Driver::OMPRT_Unknown: +break; } if (ForceStaticHostRuntime) ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D67364: [Driver] Handle default case in refactored addOpenMPRuntime
pirama updated this revision to Diff 219413. pirama added a comment. Check for OMPRT_Unknown instead of a default case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67364/new/ https://reviews.llvm.org/D67364 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp Index: clang/lib/Driver/ToolChains/CommonArgs.cpp === --- clang/lib/Driver/ToolChains/CommonArgs.cpp +++ clang/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + case Driver::OMPRT_Unknown: +break; } if (ForceStaticHostRuntime) Index: clang/lib/Driver/ToolChains/CommonArgs.cpp === --- clang/lib/Driver/ToolChains/CommonArgs.cpp +++ clang/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + case Driver::OMPRT_Unknown: +break; } if (ForceStaticHostRuntime) ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D67364: [Driver] Handle default case in refactored addOpenMPRuntime
pirama created this revision. pirama added reviewers: srhines, danalbert. Herald added a subscriber: guansong. Herald added a reviewer: jdoerfert. Herald added a project: clang. Appease failed builds (due to -Werror and -Wswitch) where OMPRT_Unknown is not handled in the switch statement (even though it's handled by the early exit). This fixes -Wswitch triggered by r371442. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D67364 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp Index: clang/lib/Driver/ToolChains/CommonArgs.cpp === --- clang/lib/Driver/ToolChains/CommonArgs.cpp +++ clang/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + default: +break; } if (ForceStaticHostRuntime) Index: clang/lib/Driver/ToolChains/CommonArgs.cpp === --- clang/lib/Driver/ToolChains/CommonArgs.cpp +++ clang/lib/Driver/ToolChains/CommonArgs.cpp @@ -525,6 +525,8 @@ case Driver::OMPRT_IOMP5: CmdArgs.push_back("-liomp5"); break; + default: +break; } if (ForceStaticHostRuntime) ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits