[all-commits] [llvm/llvm-project] c7243f: [OpenMP] Only strip runtime attributes if needed

[Joseph Huber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c7243f21d3f1b1544f6067f81b02c0af9f2ee8ab
      https://github.com/llvm/llvm-project/commit/c7243f21d3f1b1544f6067f81b02c0af9f2ee8ab
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2022-06-27 (Mon, 27 Jun 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/OpenMPOpt.cpp
    M llvm/test/Transforms/OpenMP/add_attributes.ll
    M llvm/test/Transforms/OpenMP/remove_noinline_attributes.ll

  Log Message:
  -----------
  [OpenMP] Only strip runtime attributes if needed

Summary:
Currently in OpenMPOpt we strip `noinline` attributes from runtime
functions. This is here because the device bitcode library that we link
has problems with needed definitions getting prematurely optimized out.
This is only necessary for OpenMP offloading to GPUs so we should narrow
the scope for where we spend time doing this. In the future this
shouldn't be necessary as we move to using a linked library rather than
pulling in a bitcode library in Clang.