r247222 - Revert r247218: "Fix Clang-tidy misc-use-override warnings, other minor fixes"
Hans Wennborg via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 9 17:37:18 PDT 2015
Author: hans
Date: Wed Sep 9 19:37:18 2015
New Revision: 247222
URL: http://llvm.org/viewvc/llvm-project?rev=247222&view=rev
Log:
Revert r247218: "Fix Clang-tidy misc-use-override warnings, other minor fixes"
Seems it broke the Polly build.
>From http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-fast/builds/11687/steps/compile/logs/stdio:
In file included from /home/grosser/buildslave/perf-x86_64-penryn-O3-polly-fast/llvm.src/lib/TableGen/Record.cpp:14:0:
/home/grosser/buildslave/perf-x86_64-penryn-O3-polly-fast/llvm.src/include/llvm/TableGen/Record.h:369:3: error: looser throw specifier for 'virtual llvm::TypedInit::~TypedInit()'
/home/grosser/buildslave/perf-x86_64-penryn-O3-polly-fast/llvm.src/include/llvm/TableGen/Record.h:270:11: error: overriding 'virtual llvm::Init::~Init() noexcept (true)'
Modified:
cfe/trunk/lib/Basic/Targets.cpp
cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp
cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp
cfe/trunk/lib/Driver/ToolChains.h
cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp
cfe/trunk/lib/Frontend/PCHContainerOperations.cpp
Modified: cfe/trunk/lib/Basic/Targets.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/Basic/Targets.cpp (original)
+++ cfe/trunk/lib/Basic/Targets.cpp Wed Sep 9 19:37:18 2015
@@ -30,7 +30,6 @@
#include "llvm/Support/TargetParser.h"
#include <algorithm>
#include <memory>
-
using namespace clang;
//===----------------------------------------------------------------------===//
@@ -740,7 +739,7 @@ namespace {
template <typename Target>
class WebAssemblyOSTargetInfo : public OSTargetInfo<Target> {
void getOSDefines(const LangOptions &Opts, const llvm::Triple &Triple,
- MacroBuilder &Builder) const final {
+ MacroBuilder &Builder) const override final {
// A common platform macro.
if (Opts.POSIXThreads)
Builder.defineMacro("_REENTRANT");
@@ -750,7 +749,7 @@ class WebAssemblyOSTargetInfo : public O
}
// As an optimization, group static init code together in a section.
- const char *getStaticInitSectionSpecifier() const final {
+ const char *getStaticInitSectionSpecifier() const override final {
return ".text.__startup";
}
@@ -7012,13 +7011,13 @@ private:
Features["simd128"] = true;
return TargetInfo::initFeatureMap(Features, Diags, CPU, FeaturesVec);
}
- bool hasFeature(StringRef Feature) const final {
+ bool hasFeature(StringRef Feature) const override final {
return llvm::StringSwitch<bool>(Feature)
.Case("simd128", SIMDLevel >= SIMD128)
.Default(false);
}
bool handleTargetFeatures(std::vector<std::string> &Features,
- DiagnosticsEngine &Diags) final {
+ DiagnosticsEngine &Diags) override final {
for (const auto &Feature : Features) {
if (Feature == "+simd128") {
SIMDLevel = std::max(SIMDLevel, SIMD128);
@@ -7035,7 +7034,7 @@ private:
}
return true;
}
- bool setCPU(const std::string &Name) final {
+ bool setCPU(const std::string &Name) override final {
return llvm::StringSwitch<bool>(Name)
.Case("mvp", true)
.Case("bleeding-edge", true)
@@ -7043,32 +7042,32 @@ private:
.Default(false);
}
void getTargetBuiltins(const Builtin::Info *&Records,
- unsigned &NumRecords) const final {
+ unsigned &NumRecords) const override final {
Records = BuiltinInfo;
NumRecords = clang::WebAssembly::LastTSBuiltin - Builtin::FirstTSBuiltin;
}
- BuiltinVaListKind getBuiltinVaListKind() const final {
+ BuiltinVaListKind getBuiltinVaListKind() const override final {
// TODO: Implement va_list properly.
return VoidPtrBuiltinVaList;
}
void getGCCRegNames(const char *const *&Names,
- unsigned &NumNames) const final {
+ unsigned &NumNames) const override final {
Names = nullptr;
NumNames = 0;
}
void getGCCRegAliases(const GCCRegAlias *&Aliases,
- unsigned &NumAliases) const final {
+ unsigned &NumAliases) const override final {
Aliases = nullptr;
NumAliases = 0;
}
bool
validateAsmConstraint(const char *&Name,
- TargetInfo::ConstraintInfo &Info) const final {
+ TargetInfo::ConstraintInfo &Info) const override final {
return false;
}
- const char *getClobbers() const final { return ""; }
- bool isCLZForZeroUndef() const final { return false; }
- bool hasInt128Type() const final { return true; }
+ const char *getClobbers() const override final { return ""; }
+ bool isCLZForZeroUndef() const override final { return false; }
+ bool hasInt128Type() const override final { return true; }
};
const Builtin::Info WebAssemblyTargetInfo::BuiltinInfo[] = {
Modified: cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cpp Wed Sep 9 19:37:18 2015
@@ -59,7 +59,7 @@ public:
virtual const VarDecl *getThreadIDVariable() const = 0;
/// \brief Emit the captured statement body.
- void EmitBody(CodeGenFunction &CGF, const Stmt *S) override;
+ virtual void EmitBody(CodeGenFunction &CGF, const Stmt *S) override;
/// \brief Get an LValue for the current ThreadID variable.
/// \return LValue for thread id variable. This LValue always has type int32*.
@@ -156,7 +156,7 @@ public:
return OuterRegionInfo->getContextValue();
llvm_unreachable("No context value for inlined OpenMP region");
}
- void setContextValue(llvm::Value *V) override {
+ virtual void setContextValue(llvm::Value *V) override {
if (OuterRegionInfo) {
OuterRegionInfo->setContextValue(V);
return;
@@ -229,7 +229,7 @@ public:
}
};
-} // anonymous namespace
+} // namespace
LValue CGOpenMPRegionInfo::getThreadIDVariableLValue(CodeGenFunction &CGF) {
return CGF.MakeNaturalAlignAddrLValue(
@@ -1283,7 +1283,7 @@ public:
CGF.EmitRuntimeCall(Callee, Args);
}
};
-} // anonymous namespace
+} // namespace
void CGOpenMPRuntime::emitCriticalRegion(CodeGenFunction &CGF,
StringRef CriticalName,
@@ -1831,7 +1831,7 @@ enum KmpTaskTFields {
/// \brief Function with call of destructors for private variables.
KmpTaskTDestructors,
};
-} // anonymous namespace
+} // namespace
void CGOpenMPRuntime::emitKmpRoutineEntryT(QualType KmpInt32Ty) {
if (!KmpRoutineEntryPtrTy) {
@@ -1866,7 +1866,7 @@ struct PrivateHelpersTy {
const VarDecl *PrivateElemInit;
};
typedef std::pair<CharUnits /*Align*/, PrivateHelpersTy> PrivateDataTy;
-} // anonymous namespace
+} // namespace
static RecordDecl *
createPrivatesRecordDecl(CodeGenModule &CGM,
@@ -2910,3 +2910,4 @@ void CGOpenMPRuntime::emitCancelCall(Cod
}
}
}
+
Modified: cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp (original)
+++ cfe/trunk/lib/CodeGen/ObjectFilePCHContainerOperations.cpp Wed Sep 9 19:37:18 2015
@@ -30,7 +30,6 @@
#include "llvm/Object/ObjectFile.h"
#include "llvm/Support/TargetRegistry.h"
#include <memory>
-
using namespace clang;
#define DEBUG_TYPE "pchcontainer"
@@ -140,7 +139,7 @@ public:
CodeGenOpts.SplitDwarfFile = OutputFileName;
}
- ~PCHContainerGenerator() override = default;
+ virtual ~PCHContainerGenerator() {}
void Initialize(ASTContext &Context) override {
assert(!Ctx && "initialized multiple times");
@@ -255,7 +254,7 @@ public:
}
};
-} // anonymous namespace
+} // namespace
std::unique_ptr<ASTConsumer>
ObjectFilePCHContainerWriter::CreatePCHContainerGenerator(
@@ -291,4 +290,5 @@ void ObjectFilePCHContainerReader::Extra
// As a fallback, treat the buffer as a raw AST.
StreamFile.init((const unsigned char *)Buffer.getBufferStart(),
(const unsigned char *)Buffer.getBufferEnd());
+ return;
}
Modified: cfe/trunk/lib/Driver/ToolChains.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.h?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.h (original)
+++ cfe/trunk/lib/Driver/ToolChains.h Wed Sep 9 19:37:18 2015
@@ -923,7 +923,7 @@ public:
const llvm::opt::ArgList &Args);
~SHAVEToolChain() override;
- Tool *SelectTool(const JobAction &JA) const override;
+ virtual Tool *SelectTool(const JobAction &JA) const override;
protected:
Tool *getTool(Action::ActionClass AC) const override;
@@ -960,4 +960,4 @@ private:
} // end namespace driver
} // end namespace clang
-#endif // LLVM_CLANG_LIB_DRIVER_TOOLCHAINS_H
+#endif
Modified: cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp (original)
+++ cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp Wed Sep 9 19:37:18 2015
@@ -606,7 +606,7 @@ public:
/// \brief Puts all tokens into a single line.
unsigned formatLine(const AnnotatedLine &Line, unsigned FirstIndent,
- bool DryRun) override {
+ bool DryRun) {
unsigned Penalty = 0;
LineState State = Indenter->getInitialState(FirstIndent, &Line, DryRun);
while (State.NextToken) {
@@ -629,7 +629,7 @@ public:
/// \brief Formats the line by finding the best line breaks with line lengths
/// below the column limit.
unsigned formatLine(const AnnotatedLine &Line, unsigned FirstIndent,
- bool DryRun) override {
+ bool DryRun) {
LineState State = Indenter->getInitialState(FirstIndent, &Line, DryRun);
// If the ObjC method declaration does not fit on a line, we should format
@@ -791,7 +791,7 @@ private:
llvm::SpecificBumpPtrAllocator<StateNode> Allocator;
};
-} // anonymous namespace
+} // namespace
unsigned
UnwrappedLineFormatter::format(const SmallVectorImpl<AnnotatedLine *> &Lines,
Modified: cfe/trunk/lib/Frontend/PCHContainerOperations.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/PCHContainerOperations.cpp?rev=247222&r1=247221&r2=247222&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/PCHContainerOperations.cpp (original)
+++ cfe/trunk/lib/Frontend/PCHContainerOperations.cpp Wed Sep 9 19:37:18 2015
@@ -16,7 +16,6 @@
#include "llvm/Bitcode/BitstreamReader.h"
#include "llvm/Support/raw_ostream.h"
#include "clang/Lex/ModuleLoader.h"
-
using namespace clang;
namespace {
@@ -37,7 +36,7 @@ public:
std::shared_ptr<PCHBuffer> Buffer)
: Buffer(Buffer), OS(OS) {}
- ~RawPCHContainerGenerator() override = default;
+ virtual ~RawPCHContainerGenerator() {}
void HandleTranslationUnit(ASTContext &Ctx) override {
if (Buffer->IsComplete) {
@@ -50,8 +49,7 @@ public:
Buffer->Data = std::move(Empty);
}
};
-
-} // anonymous namespace
+}
std::unique_ptr<ASTConsumer> RawPCHContainerWriter::CreatePCHContainerGenerator(
DiagnosticsEngine &Diags, const HeaderSearchOptions &HSO,
More information about the cfe-commits
mailing list