]> granicus.if.org Git - llvm/commit
Reapply "[GlobalISel] Remove the GISelAccessor API."
authorQuentin Colombet <qcolombet@apple.com>
Tue, 15 Aug 2017 22:31:51 +0000 (22:31 +0000)
committerQuentin Colombet <qcolombet@apple.com>
Tue, 15 Aug 2017 22:31:51 +0000 (22:31 +0000)
commit8e98e027842027dd9973370f111b9f0cf6137752
tree5890c62f713cd263b3471e86ee0d07e5c214d0f4
parent417caf5c7c02515698d5f9a28654973974f9cfcb
Reapply "[GlobalISel] Remove the GISelAccessor API."

This reverts commit r310425, thus reapplying r310335 with a fix for link
issue of the AArch64 unittests on Linux bots when BUILD_SHARED_LIBS is ON.

Original commit message:
[GlobalISel] Remove the GISelAccessor API.

Its sole purpose was to avoid spreading around ifdefs related to
building global-isel. Since r309990, GlobalISel is not optional anymore,
thus, we can get rid of this mechanism all together.

NFC.

----
The fix for the link issue consists in adding the GlobalISel library in
the list of dependencies for the AArch64 unittests. This dependency
comes from the use of AArch64Subtarget that needs to know how
to destruct the GISel related APIs when being detroyed.

Thanks to Bill Seurer and Ahmed Bougacha for helping me reproducing and
understand the problem.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@310969 91177308-0d34-0410-b5e6-96231b3b80d8
docs/GlobalISel.rst
include/llvm/CodeGen/GlobalISel/GISelAccessor.h [deleted file]
lib/Target/AArch64/AArch64Subtarget.cpp
lib/Target/AArch64/AArch64Subtarget.h
lib/Target/AMDGPU/AMDGPUSubtarget.cpp
lib/Target/AMDGPU/AMDGPUSubtarget.h
lib/Target/ARM/ARMSubtarget.cpp
lib/Target/ARM/ARMSubtarget.h
lib/Target/X86/X86Subtarget.cpp
lib/Target/X86/X86Subtarget.h
unittests/Target/AArch64/CMakeLists.txt