[all-commits] [llvm/llvm-project] 41a6b5: [OpenMP]Fix PR51349: Remove AlwaysInline for if re...

Joseph Huber via All-commits all-commits at lists.llvm.org
Fri Aug 6 14:53:20 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 41a6b50c25961addc04438b567ee1f4ef9e40f98
      https://github.com/llvm/llvm-project/commit/41a6b50c25961addc04438b567ee1f4ef9e40f98
  Author: Joseph Huber <jhuber6 at vols.utk.edu>
  Date:   2021-08-06 (Fri, 06 Aug 2021)

  Changed paths:
    M clang/lib/CodeGen/CGOpenMPRuntime.cpp
    A clang/test/OpenMP/parallel_if_codegen_PR51349.cpp

  Log Message:
  -----------
  [OpenMP]Fix PR51349: Remove AlwaysInline for if regions.

After D94315 we add the `NoInline` attribute to the outlined function to handle
data environments in the OpenMP if clause. This conflicted with the `AlwaysInline`
attribute added to the outlined function. for better performance in D106799.
The data environments should ideally not require NoInline, but for now this
fixes PR51349.

Reviewed By: mikerice

Differential Revision: https://reviews.llvm.org/D107649




More information about the All-commits mailing list