[clang] [llvm] [HLSL] Replace `element_type*` handles in HLSLExternalSemaSource with `__hlsl_resource_t` builtin type (PR #110079)
Helena Kotas via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 26 14:41:48 PDT 2024
================
@@ -2272,8 +2272,8 @@ TypeInfo ASTContext::getTypeInfoImpl(const Type *T) const {
#include "clang/Basic/AMDGPUTypes.def"
#define HLSL_INTANGIBLE_TYPE(Name, Id, SingletonId) case BuiltinType::Id:
#include "clang/Basic/HLSLIntangibleTypes.def"
- Width = 0;
- Align = 8;
+ Width = Target->getPointerWidth(LangAS::Default);
----------------
hekota wrote:
Yes. Conceptually we wanted the intangible types to be sizeless, but since the codegen is translating them to llvm target type that has a size, the size of the source type must be non-zero as well.
https://github.com/llvm/llvm-project/pull/110079
More information about the cfe-commits
mailing list