Fix inappropriate use of anonymous namespaces in unittests.
authorChandler Carruth <chandlerc@gmail.com>
Wed, 20 Jun 2012 08:39:27 +0000 (08:39 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Wed, 20 Jun 2012 08:39:27 +0000 (08:39 +0000)
commit1880e2d34458f67a92e26953c257f2feb334ceef
tree65b1bfc9e9ff3c6bdef79b1ac2829dc29d01629a
parent65d36f03675566ffced955874620650a8264ac80
Fix inappropriate use of anonymous namespaces in unittests.

The TEST_F macros actually declare *subclasses* of the test fixtures.
Even if they didn't we don't want them to declare external functions.
The entire unit test, including both the fixture class and the fixture
test cases should be wrapped in the anonymous namespace.

This issue was caught by the new '-Winternal-linkage-in-inline' warning.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@158798 91177308-0d34-0410-b5e6-96231b3b80d8
unittests/Support/IRBuilderTest.cpp
unittests/Transforms/Utils/Cloning.cpp