[Libomptarget][NFC] Make Libomptarget use the LLVM naming conventionLibomptarget grew out of a project that was originally not in LLVM. Aswe develop libomptarget this has led to an increasingly la
[Libomptarget][NFC] Make Libomptarget use the LLVM naming conventionLibomptarget grew out of a project that was originally not in LLVM. Aswe develop libomptarget this has led to an increasingly large clashbetween the naming conventions used. This patch fixes most of thevariable names that did not confrom to the LLVM standard, that is`VariableName` for variables and `functionName` for functions.This patch was primarily done using my editor's linting messages, ifthere are any issues I missed arising from the automation let me know.Reviewed By: saiislamDifferential Revision: https://reviews.llvm.org/D128997
show more ...
[Libomptarget] Remove global include directory from libomptargetWe used to globally include the libomptarget include directory for allprojects. This caused some conflicts with the other files name
[Libomptarget] Remove global include directory from libomptargetWe used to globally include the libomptarget include directory for allprojects. This caused some conflicts with the other files named"Debug.h". This patch changes the cmake to include these files via thetarget include instead.Reviewed By: tianshilei1992Differential Revision: https://reviews.llvm.org/D125563
[OpenMP][CMake] Clean up the CMake variable `LIBOMPTARGET_LLVM_INCLUDE_DIRS``LIBOMPTARGET_LLVM_INCLUDE_DIRS` is currently checked and included formultiple times redundantly. This patch is simply a
[OpenMP][CMake] Clean up the CMake variable `LIBOMPTARGET_LLVM_INCLUDE_DIRS``LIBOMPTARGET_LLVM_INCLUDE_DIRS` is currently checked and included formultiple times redundantly. This patch is simply a clean up.Reviewed By: jhuber6Differential Revision: https://reviews.llvm.org/D121055
[openmp] [elf_common] Fix linking against LLVM dylibThe hand-rolled linking logic in elf_common does not account forthe possibility of using LLVM dylib rather than a dozen staticlibraries. Since
[openmp] [elf_common] Fix linking against LLVM dylibThe hand-rolled linking logic in elf_common does not account forthe possibility of using LLVM dylib rather than a dozen staticlibraries. Since it does not seem to be easily convertibleto add_llvm_library, just hand-roll support for LLVM_LINK_LLVM_DYLIB.This is necessary to support stand-alone builds against installed LLVM.Differential Revision: https://reviews.llvm.org/D111038
[NFC][libomptarget] Remove redundant libelf dependency for elf_common.Differential Revision: https://reviews.llvm.org/D104549
[NFC][libomptarget] Build elf_common with PIC.Differential Revision: https://reviews.llvm.org/D104545
[NFC][libomptarget] Fixed -DLLVM_ENABLE_RUNTIMES="openmp" build.Differential Revision: https://reviews.llvm.org/D104535
[NFC][libomptarget] Reduce the dependency on libelfThis change-set removes libelf usage from elf_common part of the plugins.libelf is still used in x86_64 generic plugin code and in some plugins(
[NFC][libomptarget] Reduce the dependency on libelfThis change-set removes libelf usage from elf_common part of the plugins.libelf is still used in x86_64 generic plugin code and in some plugins(e.g. amdgpu) - these will have to be cleaned up in separate checkins.Differential Revision: https://reviews.llvm.org/D103545
[OpenMP] Take elf_common.c as a interface libraryFor now `elf_common.c` is taken as a common part included intodifferent plugin implementations directly via`#include "../../common/elf_common.c"`,
[OpenMP] Take elf_common.c as a interface libraryFor now `elf_common.c` is taken as a common part included intodifferent plugin implementations directly via`#include "../../common/elf_common.c"`, which is not a best practice. Since itis simple enough such that we don't need to create a real library for it, we justtake it as a interface library so that other targets can link it directly. Anotheradvantage of this method is, we don't need to add the folder into header searchpath which can potentially pollute the search path.VE and AMD platforms have not been tested because I don't have target machines.Reviewed By: JonChesterfieldDifferential Revision: https://reviews.llvm.org/D94443