[Mlir-commits] [llvm] [openmp] [clang] [mlir] [OpenMP] Remove `register_requires` global constructor (PR #80460)

Joseph Huber llvmlistbot at llvm.org
Mon Feb 5 05:52:19 PST 2024


================
@@ -6872,35 +6883,6 @@ void OpenMPIRBuilder::loadOffloadInfoMetadata(StringRef HostFilePath) {
   loadOffloadInfoMetadata(*M.get());
 }
 
-Function *OpenMPIRBuilder::createRegisterRequires(StringRef Name) {
----------------
jhuber6 wrote:

I encoded the fact that this is a "requires" entry into the global with `16`. Realistically it shouldn't make a difference in the runtime since as long as `16` is set and the address is null it will get counted as a requires entry. It's just a little weird that it wouldn't *just* be 16. I'll double check. Changing these entries is also on the list.

https://github.com/llvm/llvm-project/pull/80460


More information about the Mlir-commits mailing list