[clang-tools-extra] r241335 - Revert r241330. It compiled with Visual C++ 2013 and gcc 4.9.1 (mingw) but now fails the bots.
Yaron Keren
yaron.keren at gmail.com
Fri Jul 3 09:49:11 PDT 2015
Hi David,
After reverting I noticed the errors were in the unit tests. Due to an
error on my machine the unit tests were not running locally (the regression
tests did pass with VC and gcc). I asked Eugene to fix the patch so that
the unit tests will pass. I'll also make sure the unit tests run locally
and pass in the future before re-submitting the patch.
If you'd still like to look into this:
http://lab.llvm.org:8011/builders/clang-hexagon-elf/builds/28536/steps/compile/logs/stdio
llvm[4]: Compiling RecursiveASTVisitorTestExprVisitor.cpp for
Release+Asserts
build/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/ParserTest.cpp:
In function 'clang::ast_matchers::dynamic::Parser::NamedValueMap
clang::ast_matchers::dynamic::{anonymous}::getTestNamedValues()':
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/ParserTest.cpp:165:19:
error: no match for 'operator=' (operand types are
'clang::ast_matchers::dynamic::VariantValue' and 'std::string {aka
std::basic_string<char>}')
Values["nameX"] = std::string("x");
^
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/ParserTest.cpp:165:19:
note: candidate is:
In file included from
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/../../../include/clang/ASTMatchers/Dynamic/Diagnostics.h:18:0,
from
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/../../../include/clang/ASTMatchers/Dynamic/Parser.h:37,
from
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/ParserTest.cpp:11:
/var/lib/buildbot/slaves/hexagon-build-03/clang-hexagon-elf/llvm.src/tools/clang/unittests/ASTMatchers/Dynamic/../../../include/clang/ASTMatchers/Dynamic/VariantValue.h:253:17:
note: clang::ast_matchers::dynamic::VariantValue&
clang::ast_matchers::dynamic::VariantValue::operator=(const
clang::ast_matchers::dynamic::VariantValue&)
VariantValue &operator=(const VariantValue &Other);
^
...
2015-07-03 18:36 GMT+03:00 David Blaikie <dblaikie at gmail.com>:
>
>
> On Fri, Jul 3, 2015 at 2:30 AM, Yaron Keren <yaron.keren at gmail.com> wrote:
>
>> Author: yrnkrn
>> Date: Fri Jul 3 04:30:33 2015
>> New Revision: 241335
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=241335&view=rev
>> Log:
>> Revert r241330. It compiled with Visual C++ 2013 and gcc 4.9.1 (mingw)
>> but now fails the bots.
>>
>
> Got a link to the buildbot failure or copy/paste the error message(s) from
> them?
>
>
>>
>>
>> Modified:
>> clang-tools-extra/trunk/clang-modernize/LoopConvert/StmtAncestor.h
>> clang-tools-extra/trunk/clang-rename/RenamingAction.cpp
>> clang-tools-extra/trunk/clang-rename/RenamingAction.h
>> clang-tools-extra/trunk/modularize/Modularize.cpp
>> clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp
>> clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.h
>>
>> Modified:
>> clang-tools-extra/trunk/clang-modernize/LoopConvert/StmtAncestor.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/LoopConvert/StmtAncestor.h?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/clang-modernize/LoopConvert/StmtAncestor.h
>> (original)
>> +++ clang-tools-extra/trunk/clang-modernize/LoopConvert/StmtAncestor.h
>> Fri Jul 3 04:30:33 2015
>> @@ -170,9 +170,9 @@ private:
>> class DeclFinderASTVisitor :
>> public clang::RecursiveASTVisitor<DeclFinderASTVisitor> {
>> public:
>> - DeclFinderASTVisitor(std::string Name,
>> - const StmtGeneratedVarNameMap *GeneratedDecls)
>> - : Name(std::move(Name)), GeneratedDecls(GeneratedDecls),
>> Found(false) {}
>> + DeclFinderASTVisitor(const std::string &Name,
>> + const StmtGeneratedVarNameMap *GeneratedDecls) :
>> + Name(Name), GeneratedDecls(GeneratedDecls), Found(false) { }
>>
>> /// Attempts to find any usages of variables name Name in Body,
>> returning
>> /// true when it is used in Body. This includes the generated loop
>> variables
>>
>> Modified: clang-tools-extra/trunk/clang-rename/RenamingAction.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-rename/RenamingAction.cpp?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/clang-rename/RenamingAction.cpp (original)
>> +++ clang-tools-extra/trunk/clang-rename/RenamingAction.cpp Fri Jul 3
>> 04:30:33 2015
>> @@ -37,11 +37,14 @@ namespace rename {
>>
>> class RenamingASTConsumer : public ASTConsumer {
>> public:
>> - RenamingASTConsumer(StringRef NewName, StringRef PrevName,
>> + RenamingASTConsumer(const std::string &NewName,
>> + const std::string &PrevName,
>> const std::vector<std::string> &USRs,
>> - tooling::Replacements &Replaces, bool
>> PrintLocations)
>> + tooling::Replacements &Replaces,
>> + bool PrintLocations)
>> : NewName(NewName), PrevName(PrevName), USRs(USRs),
>> Replaces(Replaces),
>> - PrintLocations(PrintLocations) {}
>> + PrintLocations(PrintLocations) {
>> + }
>>
>> void HandleTranslationUnit(ASTContext &Context) override {
>> const auto &SourceMgr = Context.getSourceManager();
>> @@ -55,7 +58,7 @@ public:
>> NewCandidates.clear();
>> }
>>
>> - auto PrevNameLen = PrevName.size();
>> + auto PrevNameLen = PrevName.length();
>> if (PrintLocations)
>> for (const auto &Loc : RenamingCandidates) {
>> FullSourceLoc FullLoc(Loc, SourceMgr);
>> @@ -72,7 +75,7 @@ public:
>> }
>>
>> private:
>> - StringRef NewName, PrevName;
>> + const std::string &NewName, &PrevName;
>> const std::vector<std::string> &USRs;
>> tooling::Replacements &Replaces;
>> bool PrintLocations;
>>
>> Modified: clang-tools-extra/trunk/clang-rename/RenamingAction.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-rename/RenamingAction.h?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/clang-rename/RenamingAction.h (original)
>> +++ clang-tools-extra/trunk/clang-rename/RenamingAction.h Fri Jul 3
>> 04:30:33 2015
>> @@ -25,7 +25,7 @@ namespace rename {
>>
>> class RenamingAction {
>> public:
>> - RenamingAction(llvm::StringRef NewName, llvm::StringRef PrevName,
>> + RenamingAction(const std::string &NewName, const std::string &PrevName,
>> const std::vector<std::string> &USRs,
>> tooling::Replacements &Replaces, bool PrintLocations =
>> false)
>> : NewName(NewName), PrevName(PrevName), USRs(USRs),
>> Replaces(Replaces),
>> @@ -35,7 +35,7 @@ public:
>> std::unique_ptr<ASTConsumer> newASTConsumer();
>>
>> private:
>> - llvm::StringRef NewName, PrevName;
>> + const std::string &NewName, &PrevName;
>> const std::vector<std::string> &USRs;
>> tooling::Replacements &Replaces;
>> bool PrintLocations;
>>
>> Modified: clang-tools-extra/trunk/modularize/Modularize.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/Modularize.cpp?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/modularize/Modularize.cpp (original)
>> +++ clang-tools-extra/trunk/modularize/Modularize.cpp Fri Jul 3 04:30:33
>> 2015
>> @@ -466,9 +466,9 @@ class EntityMap : public StringMap<Small
>> public:
>> DenseMap<const FileEntry *, HeaderContents> HeaderContentMismatches;
>>
>> - void add(StringRef Name, enum Entry::EntryKind Kind, Location Loc) {
>> + void add(const std::string &Name, enum Entry::EntryKind Kind, Location
>> Loc) {
>> // Record this entity in its header.
>> - HeaderEntry HE = {Name.str(), Loc};
>> + HeaderEntry HE = { Name, Loc };
>> CurHeaderContents[Loc.File].push_back(HE);
>>
>> // Check whether we've seen this entry before.
>>
>> Modified: clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp (original)
>> +++ clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.cpp Fri Jul 3
>> 04:30:33 2015
>> @@ -627,7 +627,7 @@ void PPCallbacksTracker::appendArgument(
>>
>> // Append a double-quoted argument to the top trace item.
>> void PPCallbacksTracker::appendQuotedArgument(const char *Name,
>> - llvm::StringRef Value) {
>> + const std::string &Value) {
>> std::string Str;
>> llvm::raw_string_ostream SS(Str);
>> SS << "\"" << Value << "\"";
>>
>> Modified: clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.h?rev=241335&r1=241334&r2=241335&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.h (original)
>> +++ clang-tools-extra/trunk/pp-trace/PPCallbacksTracker.h Fri Jul 3
>> 04:30:33 2015
>> @@ -215,7 +215,7 @@ public:
>> void appendArgument(const char *Name, const clang::Module *Value);
>>
>> /// \brief Append a double-quoted argument to the top trace item.
>> - void appendQuotedArgument(const char *Name, llvm::StringRef Value);
>> + void appendQuotedArgument(const char *Name, const std::string &Value);
>>
>> /// \brief Append a double-quoted file path argument to the top trace
>> item.
>> void appendFilePathArgument(const char *Name, llvm::StringRef Value);
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150703/9af824f5/attachment.html>
More information about the cfe-commits
mailing list