Skip to content
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.

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

PR for llvm/llvm-project#54560 #187

Open
wants to merge 3 commits into
base: release/14.x
Choose a base branch
from

Commits on Apr 29, 2022

  1. [CUDA][HIP] Externalize kernels in anonymous name space

    kernels in anonymous name space needs to have unique name
    to avoid duplicate symbols.
    
    Fixes: llvm#54560
    
    Reviewed by: Artem Belevich
    
    Differential Revision: https://reviews.llvm.org/D123353
    yxsamliu committed Apr 29, 2022
    Configuration menu
    Copy the full SHA
    c949939 View commit details
    Browse the repository at this point in the history
  2. [CUDA][HIP] Externalize kernels with internal linkage

    This patch is a continuation of https://reviews.llvm.org/D123353.
    
    Not only kernels in anonymous namespace, but also template
    kernels with template arguments in anonymous namespace
    need to be externalized.
    
    To be more generic, this patch checks the linkage of a kernel
    assuming the kernel does not have __global__ attribute. If
    the linkage is internal then clang will externalize it.
    
    This patch also fixes the postfix for externalized symbol
    since nvptx does not allow '.' in symbol name.
    
    Reviewed by: Artem Belevich
    
    Differential Revision: https://reviews.llvm.org/D124189
    
    Fixes: llvm#54560
    yxsamliu committed Apr 29, 2022
    Configuration menu
    Copy the full SHA
    906395c View commit details
    Browse the repository at this point in the history

Commits on May 24, 2022

  1. Configuration menu
    Copy the full SHA
    fda1628 View commit details
    Browse the repository at this point in the history