if (!OrigFuncRange)
return llvm::createStringError(llvm::inconvertibleErrorCode(),
"Couldn't get range for function.");
- // Include template parameter list.
- if (auto *FTD = FD->getDescribedFunctionTemplate())
- OrigFuncRange->setBegin(FTD->getBeginLoc());
+ assert(!FD->getDescribedFunctionTemplate() &&
+ "Define out-of-line doesn't apply to function templates.");
// Get new begin and end positions for the qualified function definition.
unsigned FuncBegin = SM.getFileOffset(OrigFuncRange->getBegin());
Source->isOutOfLine())
return false;
+ // Bail out if this is a function template or specialization, as their
+ // definitions need to be visible in all including translation units.
+ if (auto *PT = Source->getDescribedFunctionTemplate())
+ return false;
+ if (auto *TSI = Source->getTemplateSpecializationInfo())
+ return false;
+
// Bail out in templated classes, as it is hard to spell the class name, i.e
// if the template parameter is unnamed.
if (auto *MD = llvm::dyn_cast<CXXMethodDecl>(Source)) {
EXPECT_UNAVAILABLE(R"cpp(
template <typename> struct Foo { void fo^o(){} };
)cpp");
+
+ // Not available on function templates and specializations, as definition must
+ // be visible to all translation units.
+ EXPECT_UNAVAILABLE(R"cpp(
+ template <typename> void fo^o() {};
+ template <> void fo^o<int>() {};
+ )cpp");
}
TEST_F(DefineOutlineTest, FailsWithoutSource) {
"void foo() ;",
"void foo() { return; }",
},
- // Templated function.
- {
- "template <typename T> void fo^o(T, T x) { return; }",
- "template <typename T> void foo(T, T x) ;",
- "template <typename T> void foo(T, T x) { return; }",
- },
- {
- "template <typename> void fo^o() { return; }",
- "template <typename> void foo() ;",
- "template <typename> void foo() { return; }",
- },
- // Template specialization.
- {
- R"cpp(
- template <typename> void foo();
- template <> void fo^o<int>() { return; })cpp",
- R"cpp(
- template <typename> void foo();
- template <> void foo<int>() ;)cpp",
- "template <> void foo<int>() { return; }",
- },
// Default args.
{
"void fo^o(int x, int y = 5, int = 2, int (*foo)(int) = nullptr) {}",