[libomptarget][amdgpu][nfc] Drop dead signal pool setup
authorJon Chesterfield <jonathanchesterfield@gmail.com>
Thu, 22 Jul 2021 09:29:30 +0000 (10:29 +0100)
committerJon Chesterfield <jonathanchesterfield@gmail.com>
Thu, 22 Jul 2021 09:29:32 +0000 (10:29 +0100)
commitdc1f6f8b92315fb90d9694df85ae2ce7a4a4f7e0
tree9fbbf1d8ac17688c638a4d5a73c97003cef5fa95
parentbd41136746a0b47882914cee5a8d1ac6714288d1
[libomptarget][amdgpu][nfc] Drop dead signal pool setup

This class is instantiated once in rtl.cpp before hsa_init is
called. The hsa_signal_create call therefore fails leaving the pool empty.

This signal pool is a legacy from ATMI where it was constructed after hsa_init.
Moving the state into the rtl.cpp global class disabled the initial populating
of the pool without noticeably changing performance. Just rechecked with a fix
that allocates the signals after hsa_init and that also doesn't noticeably
change performance.

This patch therefore drops the initialisation. Only change from main is to
drop a DEBUG_PRINT statement that would say the pool initial size is zero.

Reviewed By: jdoerfert

Differential Revision: https://reviews.llvm.org/D106515
openmp/libomptarget/plugins/amdgpu/impl/internal.h