[PATCH] D108258: [OpenMP] Change AAKernelInfo to ignore non-kernels

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 17 17:31:42 PDT 2021


jhuber6 created this revision.
jhuber6 added a reviewer: jdoerfert.
Herald added subscribers: ormris, guansong, hiraditya, yaxunl.
jhuber6 requested review of this revision.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.

Currently, AAKernelInfo will fail on an assertion if we attempt to run
it on a kernel without the init / deinit runtime calls. However, this
occurs for global constructors on the device. This will cause OpenMPOpt
to crash whenever global constructors are present. This patch removes
this assertion and just gives up instead.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D108258

Files:
  llvm/lib/Transforms/IPO/OpenMPOpt.cpp
  llvm/test/Transforms/OpenMP/global_constructor.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108258.367071.patch
Type: text/x-patch
Size: 5459 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210818/9f7d5038/attachment.bin>


More information about the llvm-commits mailing list