Author: Alexey Lapshin Date: 2019-12-11T12:18:46+03:00 New Revision: 21bc8958668a4dc21de8328ef856ddcf38a97da7
URL: https://github.com/llvm/llvm-project/commit/21bc8958668a4dc21de8328ef856ddcf38a97da7 DIFF: https://github.com/llvm/llvm-project/commit/21bc8958668a4dc21de8328ef856ddcf38a97da7.diff LOG: [DWARF5][SplitDwarf] Set default state for -fsplit-dwarf-inlining to be false. The -fsplit-dwarf-inlining option does not conform to DWARF5 standard. It creates children for Skeleton compilation unit. We need default behavior to be DWARF5 compatible. Thus set default state for -fsplit-dwarf-inlining into "false". Differential Revision: https://reviews.llvm.org/D71304 Added: Modified: clang/lib/Driver/ToolChains/Clang.cpp clang/test/Driver/split-debug.c Removed: ################################################################################ diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index b4621a0fcc81..4960f3e42485 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -3539,7 +3539,7 @@ static void RenderDebugOptions(const ToolChain &TC, const Driver &D, bool SplitDWARFInlining = Args.hasFlag(options::OPT_fsplit_dwarf_inlining, - options::OPT_fno_split_dwarf_inlining, true); + options::OPT_fno_split_dwarf_inlining, false); Args.ClaimAllArgs(options::OPT_g_Group); diff --git a/clang/test/Driver/split-debug.c b/clang/test/Driver/split-debug.c index 860aebb526fc..d40207d5ae3b 100644 --- a/clang/test/Driver/split-debug.c +++ b/clang/test/Driver/split-debug.c @@ -61,6 +61,9 @@ // CHECK-GMLT-WITH-SPLIT: "-split-dwarf-file" // CHECK-GMLT-WITH-SPLIT: "-split-dwarf-output" +// RUN: %clang -target x86_64-unknown-linux-gnu -g -S -### %s 2> %t +// RUN: FileCheck -check-prefix=CHECK-NOINLINE-WITHOUT-SPLIT < %t %s +// // RUN: %clang -target x86_64-unknown-linux-gnu -g -fno-split-dwarf-inlining -S -### %s 2> %t // RUN: FileCheck -check-prefix=CHECK-NOINLINE-WITHOUT-SPLIT < %t %s // @@ -79,7 +82,7 @@ // CHECK-SPLIT-WITH-NOINL: "-debug-info-kind=limited" // CHECK-SPLIT-WITH-NOINL: "-split-dwarf-output" -// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -gmlt -S -### %s 2> %t +// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -gmlt -fsplit-dwarf-inlining -S -### %s 2> %t // RUN: FileCheck -check-prefix=CHECK-GMLT-OVER-SPLIT < %t %s // // CHECK-GMLT-OVER-SPLIT: "-debug-info-kind=line-tables-only" _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
