[clang] 43f0b69 - [AMDGPU] Removal of language sensitive option for _Float16 and half('e') handling (#170443)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 9 23:42:17 PST 2025
Author: Rana Pratap Reddy
Date: 2025-12-10T13:12:12+05:30
New Revision: 43f0b6960e1816d1c1c5ec04ee2bc8c5638fb4e0
URL: https://github.com/llvm/llvm-project/commit/43f0b6960e1816d1c1c5ec04ee2bc8c5638fb4e0
DIFF: https://github.com/llvm/llvm-project/commit/43f0b6960e1816d1c1c5ec04ee2bc8c5638fb4e0.diff
LOG: [AMDGPU] Removal of language sensitive option for _Float16 and half('e') handling (#170443)
Removing the 'e' handling for the amdgcn builtins as we decided to use
_Float16 for both HIP/C++ and OpenCL
Added:
Modified:
clang/include/clang/Basic/Builtins.def
clang/lib/AST/ASTContext.cpp
Removed:
################################################################################
diff --git a/clang/include/clang/Basic/Builtins.def b/clang/include/clang/Basic/Builtins.def
index 3a5b72e20afab..b856ad145824d 100644
--- a/clang/include/clang/Basic/Builtins.def
+++ b/clang/include/clang/Basic/Builtins.def
@@ -43,7 +43,6 @@
// SJ -> sigjmp_buf
// K -> ucontext_t
// p -> pid_t
-// e -> _Float16 for HIP/C++ and __fp16 for OpenCL
// . -> "...". This may only occur at the end of the function list.
//
// Types may be prefixed with the following modifiers:
diff --git a/clang/lib/AST/ASTContext.cpp b/clang/lib/AST/ASTContext.cpp
index 13018ba8de7e7..f4aadeceb4bb7 100644
--- a/clang/lib/AST/ASTContext.cpp
+++ b/clang/lib/AST/ASTContext.cpp
@@ -12417,12 +12417,8 @@ static QualType DecodeTypeFromStr(const char *&Str, const ASTContext &Context,
// Read the base type.
switch (*Str++) {
- default: llvm_unreachable("Unknown builtin type letter!");
- case 'e':
- assert(HowLong == 0 && !Signed && !Unsigned &&
- "Bad modifiers used with 'e'!");
- Type = Context.getLangOpts().OpenCL ? Context.HalfTy : Context.Float16Ty;
- break;
+ default:
+ llvm_unreachable("Unknown builtin type letter!");
case 'x':
assert(HowLong == 0 && !Signed && !Unsigned &&
"Bad modifiers used with 'x'!");
More information about the cfe-commits
mailing list