From 13e3520d491ad9ff9270a3a05edd8e6f99878fe7 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. llvm-svn: 326116 --- clang/lib/Driver/ToolChains/Clang.cpp | 4 +--- clang/test/Driver/codeview-column-info.c | 13 ------------- 2 files changed, 1 insertion(+), 16 deletions(-) delete mode 100644 clang/test/Driver/codeview-column-info.c diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index 6ad38bc..a2248dd 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/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/clang/test/Driver/codeview-column-info.c b/clang/test/Driver/codeview-column-info.c deleted file mode 100644 index 9f2f016..0000000 --- a/clang/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.7.4