Skip to content

Commit

Permalink
[SYCL][Test] Add test for ABI-neutrality of sycl classes (#14558)
Browse files Browse the repository at this point in the history
It is difficult to understand how to detect all possible data members
which can cross ABI boundaries and cause problems because of dual ABI
issue. For now using this approach which covers most of the classes,
more than currently covered by "sycl/test/abi/layout*" tests.
Current hits are in sycl::detail CG classes and subclasses (MFileName,
MFunctinName, MKernelName and PipeName string data members). Even though
I'm not sure if those cross ABI boundaries or not, I think we have to be
conservative and fix those too.

Using -fdump-record-layouts-complete to get dump for all complete record
types, see description in https://reviews.llvm.org/D104484 for details.
  • Loading branch information
againull authored Jul 16, 2024
1 parent fe18590 commit 3254bca
Showing 1 changed file with 337 additions and 0 deletions.
Loading

0 comments on commit 3254bca

Please sign in to comment.