Skip to content
  • Joseph Huber's avatar
    Revert "[OpenMP] Remove noinline attributes in the device runtime" · fd8fd9e5
    Joseph Huber authored
    The behaviour of this patch is not great, but it has some side-effects
    that are required for OpenMPOpt to work. The problem is that when we use
    `-mlink-builtin-bitcode` we only import used symbols from the runtime.
    Then OpenMPOpt will insert calls to symbols that were not previously
    included. This patch removed this implicit behaviour as these functions
    were kept alive by the `noinline` simply because it kept calls to them
    in the module. This caused regression in some tests that relied on some
    OpenMPOpt passes without using LTO. Reverting for the LLVM15 release but
    will try to fix it more correctly on main.
    
    This reverts commit d61d72da.
    
    Fixes #56752
    
    (cherry picked from commit b08369f7)
    fd8fd9e5