Skip to content

Commit 8ef0448

Browse files
committed
Reland "[CUDA][HIP] Use the same default language std as C++""
Reland after fixing regression in lldb. Differential Revision: https://reviews.llvm.org/D155539
1 parent 9265f7c commit 8ef0448

File tree

7 files changed

+19
-24
lines changed

7 files changed

+19
-24
lines changed

clang/docs/ReleaseNotes.rst

+3
Original file line numberDiff line numberDiff line change
@@ -886,6 +886,9 @@ RISC-V Support
886886

887887
CUDA/HIP Language Changes
888888
^^^^^^^^^^^^^^^^^^^^^^^^^
889+
- Clang has been updated to align its default language standard for CUDA/HIP with
890+
that of C++. The standard has now been enhanced to gnu++17, supplanting the
891+
previously used c++14.
889892

890893
CUDA Support
891894
^^^^^^^^^^^^

clang/include/clang/Basic/LangStandards.def

-8
Original file line numberDiff line numberDiff line change
@@ -214,14 +214,6 @@ LANGSTANDARD_ALIAS_DEPR(openclcpp10, "CLC++")
214214
LANGSTANDARD_ALIAS_DEPR(openclcpp10, "CLC++1.0")
215215
LANGSTANDARD_ALIAS_DEPR(openclcpp2021, "CLC++2021")
216216

217-
// CUDA
218-
LANGSTANDARD(cuda, "cuda", CUDA, "NVIDIA CUDA(tm)",
219-
LineComment | CPlusPlus | CPlusPlus11 | CPlusPlus14 | Digraphs)
220-
221-
// HIP
222-
LANGSTANDARD(hip, "hip", HIP, "HIP",
223-
LineComment | CPlusPlus | CPlusPlus11 | CPlusPlus14 | Digraphs)
224-
225217
// HLSL
226218
LANGSTANDARD(hlsl, "hlsl",
227219
HLSL, "High Level Shader Language",

clang/lib/Basic/LangStandards.cpp

+2-4
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,6 @@ LangStandard::Kind clang::getDefaultLanguageStandard(clang::Language Lang,
5454
return LangStandard::lang_opencl12;
5555
case Language::OpenCLCXX:
5656
return LangStandard::lang_openclcpp10;
57-
case Language::CUDA:
58-
return LangStandard::lang_cuda;
5957
case Language::Asm:
6058
case Language::C:
6159
// The PS4 uses C99 as the default C standard.
@@ -66,13 +64,13 @@ LangStandard::Kind clang::getDefaultLanguageStandard(clang::Language Lang,
6664
return LangStandard::lang_gnu11;
6765
case Language::CXX:
6866
case Language::ObjCXX:
67+
case Language::CUDA:
68+
case Language::HIP:
6969
if (T.isPS())
7070
return LangStandard::lang_gnucxx14;
7171
return LangStandard::lang_gnucxx17;
7272
case Language::RenderScript:
7373
return LangStandard::lang_c99;
74-
case Language::HIP:
75-
return LangStandard::lang_hip;
7674
case Language::HLSL:
7775
return LangStandard::lang_hlsl2021;
7876
}

clang/test/CodeGenCUDA/long-double.cu

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
// RUN: -aux-triple x86_64-unknown-gnu-linux -fcuda-is-device \
77
// RUN: -emit-llvm -o - %s 2>&1 | FileCheck %s
88

9-
// CHECK: @_ZN15infinity_helperIeE5valueE = {{.*}} double 0x47EFFFFFD586B834, align 8
9+
// CHECK: @_ZN15infinity_helperIeE5valueE = {{.*}} double 0x47EFFFFFD586B834,{{.*}} align 8
1010
// CHECK: @size = {{.*}} i32 8
1111

1212
#include "Inputs/cuda.h"

clang/test/Driver/unknown-std.cpp

+4-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,10 @@
44

55
// RUN: not %clang %s -std=foobar -c 2>&1 | FileCheck --match-full-lines %s
66
// RUN: not %clang -x objective-c++ %s -std=foobar -c 2>&1 | FileCheck --match-full-lines %s
7-
// RUN: not %clang -x cuda -nocudainc -nocudalib --cuda-path=%S/Inputs/CUDA/usr/local/cuda %s -std=foobar -c 2>&1 | FileCheck --match-full-lines --check-prefix=CHECK --check-prefix=CUDA %s
7+
// RUN: not %clang -x cuda -nocudainc -nocudalib --cuda-path=%S/Inputs/CUDA/usr/local/cuda \
8+
// RUN: %s -std=foobar -c 2>&1 | FileCheck --match-full-lines %s
9+
// RUN: not %clang -x hip -nocudainc -nocudalib %s -std=foobar -c 2>&1 \
10+
// RUN: | FileCheck --match-full-lines %s
811

912
// CHECK: error: invalid value 'foobar' in '-std=foobar'
1013
// CHECK-NEXT: note: use 'c++98' or 'c++03' for 'ISO C++ 1998 with amendments' standard
@@ -21,7 +24,6 @@
2124
// CHECK-NEXT: note: use 'gnu++23' for 'ISO C++ 2023 DIS with GNU extensions' standard
2225
// CHECK-NEXT: note: use 'c++2c' or 'c++26' for 'Working draft for C++2c' standard
2326
// CHECK-NEXT: note: use 'gnu++2c' or 'gnu++26' for 'Working draft for C++2c with GNU extensions' standard
24-
// CUDA-NEXT: note: use 'cuda' for 'NVIDIA CUDA(tm)' standard
2527

2628
// Make sure that no other output is present.
2729
// CHECK-NOT: {{^.+$}}

clang/test/Preprocessor/lang-std.cpp

+7-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
// UNSUPPORTED: target={{.*-(ps4|ps5)}}
21
/// Test default standards.
3-
/// PS4/PS5 default to gnu++14.
2+
/// CUDA/HIP uses the same default standards as C++.
43

5-
// RUN: %clang_cc1 -dM -E %s | FileCheck --check-prefix=CXX17 %s
6-
// RUN: %clang_cc1 -dM -E -x cuda %s | FileCheck --check-prefix=CXX14 %s
7-
// RUN: %clang_cc1 -dM -E -x hip %s | FileCheck --check-prefix=CXX14 %s
4+
// RUN: %clang_cc1 -dM -E %s | grep __cplusplus >%T-cpp-std.txt
5+
// RUN: %clang_cc1 -dM -E -x cuda %s | grep __cplusplus >%T-cuda-cuda.txt
6+
// RUN: %clang_cc1 -dM -E -x hip %s | grep __cplusplus >%T-hip-std.txt
7+
// RUN: diff %T-cpp-std.txt %T-cuda-cuda.txt
8+
// RUN: diff %T-cpp-std.txt %T-hip-std.txt
89

910
// RUN: %clang_cc1 -dM -E -x cuda -std=c++14 %s | FileCheck --check-prefix=CXX14 %s
11+
// RUN: %clang_cc1 -dM -E -x cuda -std=c++17 %s | FileCheck --check-prefix=CXX17 %s
1012
// RUN: %clang_cc1 -dM -E -x hip -std=c++98 %s | FileCheck --check-prefix=CXX98 %s
1113

1214
// CXX98: #define __cplusplus 199711L

lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp

+2-4
Original file line numberDiff line numberDiff line change
@@ -485,9 +485,6 @@ static void ParseLangArgs(LangOptions &Opts, InputKind IK, const char *triple) {
485485
case clang::Language::OpenCLCXX:
486486
LangStd = LangStandard::lang_openclcpp10;
487487
break;
488-
case clang::Language::CUDA:
489-
LangStd = LangStandard::lang_cuda;
490-
break;
491488
case clang::Language::Asm:
492489
case clang::Language::C:
493490
case clang::Language::ObjC:
@@ -497,8 +494,9 @@ static void ParseLangArgs(LangOptions &Opts, InputKind IK, const char *triple) {
497494
case clang::Language::ObjCXX:
498495
LangStd = LangStandard::lang_gnucxx98;
499496
break;
497+
case clang::Language::CUDA:
500498
case clang::Language::HIP:
501-
LangStd = LangStandard::lang_hip;
499+
LangStd = LangStandard::lang_gnucxx17;
502500
break;
503501
case clang::Language::HLSL:
504502
LangStd = LangStandard::lang_hlsl;

0 commit comments

Comments
 (0)