Skip to content

Commit 3204c9d

Browse files
committed
Adopt suggestion for general case of -gheterogeneous-dwarf
1 parent 9ce2e74 commit 3204c9d

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

clang/lib/Driver/ToolChains/Clang.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4975,8 +4975,7 @@ renderDebugOptions(const ToolChain &TC, const Driver &D, const llvm::Triple &T,
49754975
if (EmitDwarfForAMDGCN)
49764976
CmdArgs.append({"-mllvm", "-amdgpu-spill-cfi-saved-regs"});
49774977
if (Arg *A = Args.getLastArg(options::OPT_gheterogeneous_dwarf_EQ)) {
4978-
if (StringRef(A->getValue()) == "diexpr" && T.isSPIRV() &&
4979-
T.getVendor() == llvm::Triple::AMD)
4978+
if (StringRef(A->getValue()) == "diexpr")
49804979
D.Diag(clang::diag::err_drv_unsupported_opt_with_suggestion)
49814980
<< A->getAsString(Args) << "-gheterogeneous-dwarf=diexpression";
49824981
A->render(Args, CmdArgs);

clang/test/Driver/amdgpu-debug.cl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444

4545
// Specifically, check for failure with previously-valid value diexpr
4646
// RUN: not %clang -target amdgcn-amd-amdhsa -x cl -c -nogpuinc -nogpulib -emit-llvm -g -gheterogeneous-dwarf=diexpr %s 2>&1 | FileCheck -check-prefix=CHECK-DIEXPR %s
47-
// CHECK-DIEXPR: error: invalid value
47+
// CHECK-DIEXPR: error: unsupported option '-gheterogeneous-dwarf=diexpr'; did you mean '-gheterogeneous-dwarf=diexpression'?
4848

4949
// Check that =diexpression is implied by -g + spirv
5050
// RUN: %clang -### -target spirv64-amd-amdhsa -x cl -c -nogpuinc -nogpulib -emit-llvm -g %s 2>&1 | FileCheck -check-prefix=CHECK-SPIRV %s

0 commit comments

Comments
 (0)