Skip to content

Commit

Permalink
Prevent further errors of omission when adding backend names.
Browse files Browse the repository at this point in the history
Differential Revision: http://reviews.llvm.org/D9441

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@236865 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
snuglas committed May 8, 2015
1 parent 405e5f2 commit d7e20e7
Show file tree
Hide file tree
Showing 2 changed files with 45 additions and 0 deletions.
2 changes: 2 additions & 0 deletions unittests/Support/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
set(LLVM_LINK_COMPONENTS
${LLVM_TARGETS_TO_BUILD}
Support
)

Expand Down Expand Up @@ -37,6 +38,7 @@ add_llvm_unittest(SupportTests
StreamingMemoryObject.cpp
StringPool.cpp
SwapByteOrderTest.cpp
TargetRegistry.cpp
ThreadLocalTest.cpp
TimeValueTest.cpp
UnicodeTest.cpp
Expand Down
43 changes: 43 additions & 0 deletions unittests/Support/TargetRegistry.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
//===- unittests/Support/TargetRegistry.cpp - -----------------------------===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//

#include "llvm/Support/TargetRegistry.h"
#include "llvm/Support/TargetSelect.h"
#include "gtest/gtest.h"

using namespace llvm;

namespace {

TEST(TargetRegistry, TargetHasArchType) {
// Presence of at least one target will be asserted when done with the loop,
// else this would pass by accident if InitializeAllTargetInfos were omitted.
int Count = 0;

llvm::InitializeAllTargetInfos();

llvm::TargetRegistry RegistryRoot;
for (const auto &Target : RegistryRoot) {
StringRef Name = Target.getName();
// There is really no way (at present) to ask a Target whether it targets
// a specific architecture, because the logic for that is buried in a
// predicate.
// We can't ask the predicate "Are you a function that always returns
// false?"
// So given that the cpp backend truly has no target arch, it is skipped.
if (Name != "cpp") {
Triple::ArchType Arch = Triple::getArchTypeForLLVMName(Name);
EXPECT_NE(Arch, Triple::UnknownArch);
++Count;
}
}
ASSERT_NE(Count, 0);
}

} // end namespace

0 comments on commit d7e20e7

Please sign in to comment.