[all-commits] [llvm/llvm-project] d8e0ae: [SYCL] Fix __builtin_sycl_unique_stable_name to wo...

Erich Keane via All-commits all-commits at lists.llvm.org
Wed Jun 2 13:16:32 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d8e0ae9a76a62bdc6117630d59bf9967ac9bb4ea
      https://github.com/llvm/llvm-project/commit/d8e0ae9a76a62bdc6117630d59bf9967ac9bb4ea
  Author: Erich Keane <erich.keane at intel.com>
  Date:   2021-06-02 (Wed, 02 Jun 2021)

  Changed paths:
    M clang/lib/AST/ASTContext.cpp
    A clang/test/CodeGenSYCL/unique_stable_name_windows_diff.cpp

  Log Message:
  -----------
  [SYCL] Fix __builtin_sycl_unique_stable_name to work on windows/spir

In the case where the device is an itanium target, and the host is a
windows target, we were getting the names wrong, since in the itanium
case we filter by lambda-signature.

The fix is to always filter by the signature rather than just on
non-windows builds. I considered doing the reverse (that is, checking
the aux-triple), but doing so would result in duplicate lambda mangling
numbers (from linux reusing the same number for different signatures).




More information about the All-commits mailing list