[PATCH] D156423: [OMPIRBuilder] Fix module finalization

[Dominik Adamski via Phabricator via cfe-commits]

domada created this revision.
domada added reviewers: jsjodin, skatrak, agozillon, kiranchandramohan, kiranktp, NimishMishra, TIFitis, raghavendhra, dpalermo, jdoerfert.
domada added a project: OpenMP.
Herald added subscribers: gysit, Dinistro, bviyer, Moerafaat, zero9178, bzcheeseman, awarzynski, sdasgup3, wenzhicui, wrengr, ormris, cota, teijeong, rdzhabarov, tatianashp, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, mgester, arpith-jacob, antiagainst, shauheen, rriddle, mehdi_amini, hiraditya.
Herald added a reviewer: ftynse.
Herald added a reviewer: dcaballe.
Herald added a project: All.
domada requested review of this revision.
Herald added subscribers: llvm-commits, cfe-commits, jplehr, sstefan1, stephenneuendorffer, nicolasvasilache.
Herald added projects: clang, MLIR, LLVM.

OMP offload module metadata should be created only when module generation is finalized. If we finalize LLVM IR function we should not create omp offload metadata.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D156423

Files:
  clang/lib/CodeGen/CodeGenFunction.cpp
  clang/test/OpenMP/irbuilder_omp_offload_metadata.c
  llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
  llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
  llvm/lib/Transforms/IPO/OpenMPOpt.cpp
  llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
  mlir/lib/Target/LLVMIR/ModuleTranslation.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156423.544721.patch
Type: text/x-patch
Size: 18469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230727/5d842b1c/attachment-0001.bin>