[Openmp-commits] [PATCH] D131309: [Libomptarget] Add utility functions for loading an ELF symbol by name

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Sep 7 05:02:17 PDT 2022

JonChesterfield added a comment.

Implementation looks reasonable. Extra const annotations are fine. I'm not totally convinced implementation complexity has gone down - we're down 70 lines and up 210 - what're the users of the second format of hashtable, and of the linear scan? I'm guessing cuda?

Comment at: openmp/libomptarget/plugins/amdgpu/src/rtl.cpp:1617
+  if (!SymOrErr) {
+    REPORT("Failed ELF lookup: %s\n", toString(SymOrErr.takeError()).c_str());
     return 1;
REPORT -> DP for consistency with the rest of the plugin I think

Comment at: openmp/libomptarget/plugins/common/elf_common/ELFSymbols.cpp:135
+  }
+  return nullptr;
So one of these is presumably dead, since the only caller is amdgpu. Does cuda use the other sort of hash table? If not, suggest we delete the unused code path for now

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list