[Openmp-commits] [PATCH] D130111: [OpenMP] Expose the state in the header to allow non-lto optimizations

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jul 19 12:25:36 PDT 2022


jdoerfert created this revision.
jdoerfert added a reviewer: jhuber6.
Herald added subscribers: guansong, bollu, inglorion, yaxunl.
Herald added a project: All.
jdoerfert requested review of this revision.
Herald added a subscriber: sstefan1.
Herald added a project: OpenMP.

We used to inline the `lookup` calls such that the runtime had "known"
access offsets when it was shipped. With the new static library build it
doesn't as the lookup is an indirection we cannot look through. This
should help us optimize the code better until we can do LTO for the
runtime again.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D130111

Files:
  openmp/libomptarget/DeviceRTL/include/State.h
  openmp/libomptarget/DeviceRTL/src/State.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130111.445907.patch
Type: text/x-patch
Size: 13678 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20220719/fa2ff706/attachment.bin>


More information about the Openmp-commits mailing list