Skip to content

[SYCL] Add clang support for code_location in KernelInfo #5335

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bader merged 23 commits into from
Feb 4, 2022
Merged

[SYCL] Add clang support for code_location in KernelInfo #5335

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
b4f196f
[SYCL] Add clang support for KernelInfo code_location() method
schittir Jan 6, 2022
4fada17
Fix NDEBUG typo
schittir Jan 19, 2022
3bc31c7
Populate code_location members
schittir Jan 19, 2022
457b97b
Populate fields
schittir Jan 20, 2022
5ed0c13
Make changes per Erich's comments
schittir Jan 25, 2022
114d8cd
Populate "FuncionName" and change the test
schittir Jan 27, 2022
7be77e7
Delete int_header2.cpp
schittir Jan 27, 2022
622d27a
Update FunctionName and test
schittir Jan 27, 2022
a04d561
Fix format
schittir Jan 27, 2022
d0007da
Tidying up
schittir Jan 27, 2022
e7e29b9
Remove extra space
schittir Jan 27, 2022
08a8d15
Remove temporary variables and path information from filename
schittir Jan 28, 2022
c848a82
Fix test and add more cases
schittir Jan 28, 2022
7201467
Remove comments
schittir Jan 28, 2022
338bff0
Fix format
schittir Jan 29, 2022
99242c8
Add NDEBUG cases
schittir Jan 30, 2022
c067405
Remove NDEBUG from kernel_desc.hpp
schittir Jan 31, 2022
4c59038
Fix quotes
schittir Jan 31, 2022
2b10cbf
Emit fully qualified name
schittir Feb 2, 2022
14124ff
Add an -fsycl-is-host RUN line with the int header as FileCheck input
schittir Feb 2, 2022
16d6af3
Emit NDEBUG case to integration header
schittir Feb 2, 2022
190a44d
Undo accidental deletion of __SYCL_DLL_LOCAL
schittir Feb 2, 2022
76a77f4
Add Functor test case
schittir Feb 3, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion clang/include/clang/Sema/Sema.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,18 @@ class SYCLIntegrationHeader {
/// Whether this kernel is an ESIMD one.
bool IsESIMDKernel;

/// Kernel source file name.
std::string FileName;

/// Kernel location function name.
std::string FunctionName;

/// Kernel location - line number.
unsigned LineNumber;

/// Kernel location - column number.
unsigned ColumnNumber;

/// Descriptor of kernel actual parameters.
SmallVector<KernelParamDesc, 8> Params;

Expand All @@ -405,7 +417,9 @@ class SYCLIntegrationHeader {
KernelDesc(const FunctionDecl *SyclKernel, QualType NameType,
SourceLocation KernelLoc, bool IsESIMD, bool IsUnnamedKernel)
: SyclKernel(SyclKernel), NameType(NameType), KernelLocation(KernelLoc),
IsESIMDKernel(IsESIMD), IsUnnamedKernel(IsUnnamedKernel) {}
IsESIMDKernel(IsESIMD), IsUnnamedKernel(IsUnnamedKernel),// {}
LineNumber(KernelLocation.getPresumedLoc().getLine()), //getLineNumber()),
ColumnNumber(KernelLocation.getPresumedLoc().getColumn()) {} //ColumnNumber()) {}

void updateKernelNames(StringRef Name, StringRef StableName) {
this->Name = Name.str();
Expand Down
12 changes: 12 additions & 0 deletions clang/lib/Sema/SemaSYCL.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4707,6 +4707,18 @@ void SYCLIntegrationHeader::emit(raw_ostream &O) {
O << " __SYCL_DLL_LOCAL\n";
O << " static constexpr bool isESIMD() { return " << K.IsESIMDKernel
<< "; }\n";
O << " __SYCL_DLL_LOCAL\n";
O << " static constexpr const char* getFileName() { return " << K.FileName
<< "\"; }\n";
O << " __SYCL_DLL_LOCAL\n";
O << " static constexpr const char* getFunctionName() { return "
<< K.FunctionName << "\"; }\n";
O << " __SYCL_DLL_LOCAL\n";
O << " static constexpr unsigned getLineNumber() { return " << K.LineNumber
<< "\"; }\n";
O << " __SYCL_DLL_LOCAL\n";
O << " static constexpr unsigned getColumnNumber() { return "
<< K.ColumnNumber << "\"; }\n";
O << "};\n";
CurStart += N;
}
Expand Down
2 changes: 2 additions & 0 deletions clang/test/CodeGenSYCL/int_header1.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,5 +190,7 @@ int main() {
KernelInfo<class nm1::KernelName8<nm1::nm2::C>>::getName();
KernelInfo<class TmplClassInAnonNS<class ClassInAnonNS>>::getName();
KernelInfo<class nm1::KernelName9<char>>::getName();
// KernelInfo<class KernelName>::codeLocation();
// add code location KernelInfo methods
#endif //__SYCL_DEVICE_ONLY__
}
21 changes: 21 additions & 0 deletions sycl/include/CL/sycl/detail/kernel_desc.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@ template <auto &SpecName> const char *get_spec_constant_symbolic_ID_impl();
template <auto &SpecName> const char *get_spec_constant_symbolic_ID();
#endif

//template<typename CodeLocation> const detail::ContextImplPtr *detail::code_location();
// { return ""; }
#ifndef __SYCL_UNNAMED_LAMBDA__
template <class KernelNameType> struct KernelInfo {
static constexpr unsigned getNumParams() { return 0; }
Expand All @@ -75,6 +77,13 @@ template <class KernelNameType> struct KernelInfo {
}
static constexpr const char *getName() { return ""; }
static constexpr bool isESIMD() { return 0; }
#ifndef DNDEBUG
static constexpr const char *getFileName() { return ""; }
static constexpr const char *getFunctionName() { return ""; }
static constexpr unsigned getLineNumber() { return 0; }
static constexpr unsigned getColumnNumber() { return 0; }
#endif
Comment thread
schittir marked this conversation as resolved.
Outdated

};
#else
template <char...> struct KernelInfoData {
Expand All @@ -85,6 +94,12 @@ template <char...> struct KernelInfoData {
}
static constexpr const char *getName() { return ""; }
static constexpr bool isESIMD() { return 0; }
#ifndef DNDEBUG
static constexpr const char *getFileName() { return ""; }
static constexpr const char *getFunctionName() { return ""; }
static constexpr unsigned getLineNumber() { return 0; }
static constexpr unsigned getColumnNumber() { return 0; }
#endif
};

// C++14 like index_sequence and make_index_sequence
Expand Down Expand Up @@ -123,6 +138,12 @@ template <class KernelNameType> struct KernelInfo {
}
static constexpr const char *getName() { return SubKernelInfo::getName(); }
static constexpr bool isESIMD() { return SubKernelInfo::isESIMD(); }
#ifndef DNDEBUG
static constexpr const char *getFileName() { return ""; }
static constexpr const char *getFunctionName() { return ""; }
static constexpr unsigned getLineNumber() { return 0; }
static constexpr unsigned getColumnNumber() { return 0; }
#endif
};
#endif //__SYCL_UNNAMED_LAMBDA__

Expand Down