Correct symbol visibility issues found by ubsan testing

This commit is contained in:
Robert Maynard 2019-09-11 08:30:49 -04:00
parent b95b7d6109
commit c4e1262e14
4 changed files with 19 additions and 3 deletions

@ -30,7 +30,7 @@ namespace internal
/// Before scheduling worklets, the global array should be cleared to have no
/// error. This can only be reliably done by the device adapter.
///
class VTKM_NEVER_EXPORT ErrorMessageBuffer
class VTKM_ALWAYS_EXPORT ErrorMessageBuffer
{
public:
VTKM_EXEC_CONT ErrorMessageBuffer()

@ -134,7 +134,7 @@ void VTKM_NEVER_EXPORT FunctorTiling3DExecute(void* f,
//
// Note: The worklet and invocation must have a lifetime that is at least
// as long as the Task
class VTKM_ALWAYS_EXPORT TaskTiling1D : public vtkm::exec::TaskBase
class VTKM_NEVER_EXPORT TaskTiling1D : public vtkm::exec::TaskBase
{
public:
TaskTiling1D()
@ -226,7 +226,7 @@ protected:
//
// Note: The worklet and invocation must have a lifetime that is at least
// as long as the Task
class VTKM_ALWAYS_EXPORT TaskTiling3D : public vtkm::exec::TaskBase
class VTKM_NEVER_EXPORT TaskTiling3D : public vtkm::exec::TaskBase
{
public:
TaskTiling3D()

@ -19,6 +19,10 @@
#define diy vtkmdiy // mangle namespace diy (see below comments)
#endif
#if defined(VTKM_CLANG) || defined(VTKM_GCC)
#pragma GCC visibility push(default)
#endif
// clang-format off
VTKM_THIRDPARTY_PRE_INCLUDE
#include VTKM_DIY_INCLUDE(assigner.hpp)
@ -35,6 +39,10 @@ VTKM_THIRDPARTY_PRE_INCLUDE
VTKM_THIRDPARTY_POST_INCLUDE
// clang-format on
#if defined(VTKM_CLANG) || defined(VTKM_GCC)
#pragma GCC visibility pop
#endif
// When using an external DIY
// We need to alias the diy namespace to
// vtkmdiy so that VTK-m uses it properly

@ -19,6 +19,10 @@
#define diy vtkmdiy // mangle namespace diy (see below comments)
#endif
#if defined(VTKM_CLANG) || defined(VTKM_GCC)
#pragma GCC visibility push(default)
#endif
// clang-format off
VTKM_THIRDPARTY_PRE_INCLUDE
#include VTKM_DIY_INCLUDE(serialization.hpp)
@ -26,6 +30,10 @@ VTKM_THIRDPARTY_PRE_INCLUDE
VTKM_THIRDPARTY_POST_INCLUDE
// clang-format on
#if defined(VTKM_CLANG) || defined(VTKM_GCC)
#pragma GCC visibility pop
#endif
// When using an external DIY
// We need to alias the diy namespace to
// vtkmdiy so that VTK-m uses it properly