Skip to content

Commit 21bc895

Browse files
committed
[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
1 parent f7c8ace commit 21bc895

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3539,7 +3539,7 @@ static void RenderDebugOptions(const ToolChain &TC, const Driver &D,
35393539

35403540
bool SplitDWARFInlining =
35413541
Args.hasFlag(options::OPT_fsplit_dwarf_inlining,
3542-
options::OPT_fno_split_dwarf_inlining, true);
3542+
options::OPT_fno_split_dwarf_inlining, false);
35433543

35443544
Args.ClaimAllArgs(options::OPT_g_Group);
35453545

clang/test/Driver/split-debug.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,9 @@
6161
// CHECK-GMLT-WITH-SPLIT: "-split-dwarf-file"
6262
// CHECK-GMLT-WITH-SPLIT: "-split-dwarf-output"
6363

64+
// RUN: %clang -target x86_64-unknown-linux-gnu -g -S -### %s 2> %t
65+
// RUN: FileCheck -check-prefix=CHECK-NOINLINE-WITHOUT-SPLIT < %t %s
66+
//
6467
// RUN: %clang -target x86_64-unknown-linux-gnu -g -fno-split-dwarf-inlining -S -### %s 2> %t
6568
// RUN: FileCheck -check-prefix=CHECK-NOINLINE-WITHOUT-SPLIT < %t %s
6669
//
@@ -79,7 +82,7 @@
7982
// CHECK-SPLIT-WITH-NOINL: "-debug-info-kind=limited"
8083
// CHECK-SPLIT-WITH-NOINL: "-split-dwarf-output"
8184

82-
// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -gmlt -S -### %s 2> %t
85+
// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -gmlt -fsplit-dwarf-inlining -S -### %s 2> %t
8386
// RUN: FileCheck -check-prefix=CHECK-GMLT-OVER-SPLIT < %t %s
8487
//
8588
// CHECK-GMLT-OVER-SPLIT: "-debug-info-kind=line-tables-only"

0 commit comments

Comments
 (0)