From 8890f6aaa0d9fc9cf8515495a1bb1dcf7ba4ff1a Mon Sep 17 00:00:00 2001 From: Zachary Turner Date: Mon, 26 Feb 2018 19:51:29 +0000 Subject: [PATCH] Revert "Emit proper CodeView when -gcodeview is passed without the cl driver." This reverts commit e17911006548518634fad66bb8648bcad49a1d64. This is failing on ASAN bots because asan expects column info, and it's also failing on some linux bots for unknown reasons which i need to investigate. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@326116 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Driver/ToolChains/Clang.cpp | 4 +--- test/Driver/codeview-column-info.c | 13 ------------- 2 files changed, 1 insertion(+), 16 deletions(-) delete mode 100644 test/Driver/codeview-column-info.c diff --git a/lib/Driver/ToolChains/Clang.cpp b/lib/Driver/ToolChains/Clang.cpp index 6ad38bc0e4..a2248ddaa9 100644 --- a/lib/Driver/ToolChains/Clang.cpp +++ b/lib/Driver/ToolChains/Clang.cpp @@ -2968,7 +2968,7 @@ static void RenderDebugOptions(const ToolChain &TC, const Driver &D, // Forward -gcodeview. EmitCodeView might have been set by CL-compatibility // argument parsing. - if (EmitCodeView) { + if (Args.hasArg(options::OPT_gcodeview) || EmitCodeView) { // DWARFVersion remains at 0 if no explicit choice was made. CmdArgs.push_back("-gcodeview"); } else if (DWARFVersion == 0 && @@ -3567,8 +3567,6 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, types::ID InputType = Input.getType(); if (D.IsCLMode()) AddClangCLArgs(Args, InputType, CmdArgs, &DebugInfoKind, &EmitCodeView); - else - EmitCodeView = Args.hasArg(options::OPT_gcodeview); const Arg *SplitDWARFArg = nullptr; RenderDebugOptions(getToolChain(), D, RawTriple, Args, EmitCodeView, diff --git a/test/Driver/codeview-column-info.c b/test/Driver/codeview-column-info.c deleted file mode 100644 index 9f2f016a5f..0000000000 --- a/test/Driver/codeview-column-info.c +++ /dev/null @@ -1,13 +0,0 @@ -// Check that -dwarf-column-info does not get added to the cc1 line: -// 1) When -gcodeview is present via the clang or clang++ driver -// 2) When /Z7 is present via the cl driver. - -// RUN: %clang -### -c -g -gcodeview %s 2> %t1 -// RUN: FileCheck < %t1 %s -// RUN: %clangxx -### -c -g -gcodeview %s 2> %t2 -// RUN: FileCheck < %t2 %s -// RUN: %clang_cl -### /c /Z7 %s 2> %t2 -// RUN: FileCheck < %t2 %s - -// CHECK: "-cc1" -// CHECK-NOT: "-dwarf-column-info" -- 2.40.0