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

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 7 05:11:25 PDT 2022


JonChesterfield added inline comments.


================
Comment at: openmp/libomptarget/plugins/common/elf_common/ELFSymbols.cpp:135
+  }
+
+  return nullptr;
----------------
jhuber6 wrote:
> JonChesterfield wrote:
> > 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
> I don't think CUDA even uses a hash table as it's some weird executable, for that we will need the exhaustive search below. I mostly kept this because we may have more users in the future and it was easier to just include it all since I already wrote it.
Yeah, none of that. Delete it please.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131309/new/

https://reviews.llvm.org/D131309



More information about the llvm-commits mailing list