• Daniel Bevenius's avatar
    [test] Suppress subobject-linkage warnings · 3705048d
    Daniel Bevenius authored
    Currently there are a number of -Wsubobject-linkage warnings when
    compiling with gcc (formatted to fit 72 character lines):
    
    In file included from
    ...
    from ../../testing/gtest/include/gtest/gtest.h:10,
    from ../../testing/gtest-support.h:8,
    from ../../test/unittests/test-utils.h:20,
    from ../../test/unittests/compiler/backend/
      instruction-selector-unittest.h:15,
    from ../../test/unittests/compiler/x64/
      instruction-selector-x64-unittest.cc:9:
    ../../third_party/googletest/src/googletest/include/gtest/internal/
    gtest-param-util.h:
    In instantiation of ‘class
    testing::internal::ParameterizedTestFactory<v8::internal::compiler::
    InstructionSelectorChangeInt32ToInt64Test_ \
    ChangeInt32ToInt64WithLoad_Test>’:
    ../../third_party/googletest/src/googletest/include/gtest/internal/
    gtest-param-util.h:439:12:   required from
    ‘testing::internal::TestFactoryBase*
    testing::internal::TestMetaFactory<TestSuite>::CreateTestFactory(
      testing::internal::TestMetaFactory<TestSuite>::ParamType)
    [with
    TestSuite = v8::internal::compiler::
    InstructionSelectorChangeInt32ToInt64Test_ \
    ChangeInt32ToInt64WithLoad_Test;
    testing::internal::TestMetaFactory<TestSuite>::ParamType =
      v8::internal::compiler::{anonymous}::LoadWithToInt64Extension]’
    ../../third_party/googletest/src/googletest/include/gtest/internal/
      gtest-param-util.h:438:20:   required from here
    ../../third_party/googletest/src/googletest/include/gtest/internal/
      gtest-param-util.h:394:7: warning:
    ‘testing::internal::ParameterizedTestFactory<
    v8::internal::compiler::
    InstructionSelectorChangeInt32ToInt64Test_ \
    ChangeInt32ToInt64WithLoad_Test >’ has a field
    ‘testing::internal::ParameterizedTestFactory<
    v8::internal::compiler::
    InstructionSelectorChangeInt32ToInt64Test_ \
    ChangeInt32ToInt64WithLoad_Test>::parameter_’ whose type uses the
    anonymous namespace [-Wsubobject-linkage]
      394 | class ParameterizedTestFactory : public TestFactoryBase {
          |       ^~~~~~~~~~~~~~~~~~~~~~~~
    
    This commit moves the parameterized tests in question into the
    anonymous namespace to avoid the warnings.
    
    Change-Id: I9c4a8bd9f4e225ed14ab64f5433d5f5c102e01a1
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2418723Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Commit-Queue: Andreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#70482}
    3705048d
instruction-selector-x64-unittest.cc 70.4 KB