[Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 (swift 4.2) #646

Report

[FAILURE] oss-swift-4.2-incremental-RA-linux-ubuntu-16_04 [#646]

Build URL:
https://ci.swift.org/job/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/646/
Project:
oss-swift-4.2-incremental-RA-linux-ubuntu-16_04
Date of build:
Tue, 17 Apr 2018 13:37:22 -0500
Build duration:
20 min

Identified problems:

  • Compile Error: This build failed because of a compile error. Below is a list of all errors in the build log:

Changes

  • Commit ef7cc62640dc688e9277067bffed9137b221b7c3 by kremenek:
    Add mailmap for git history.

    • add: .mailmap
  • Commit d515a622008753d0b9c5df347216bd2de1591529 by mgottesman:
    [ADT] Simplify and optimize StringSwitch

    • edit: include/llvm/ADT/StringSwitch.h
    • edit: unittests/ADT/StringSwitchTest.cpp
    • edit: include/llvm/ADT/StringRef.h
  • Commit f6ae0f82b006d9c785e164635ae842e9f1e2059a by mgottesman:
    [ADT] NFC: Fix bogus StringSwitch rule-of-five boilerplate

    • edit: include/llvm/ADT/StringSwitch.h
  • Commit ea4226c6e773003ded00be1e7566272a0874689c by mgottesman:
    Fix for LLVM r326109

    • edit: lib/Basic/SourceManager.cpp
  • Commit fa45d4d763d26253f545d8890cba56325b278af8 by kremenek:
    Add mailmap for git history.

    • add: .mailmap

Michael G, is this related to your adoption of StringSwitch?

/home/buildnode/jenkins/workspace/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/swift/lib/RemoteAST/RemoteAST.cpp:590:13: error: no viable conversion from 'const char [4]' to 'llvm::StringLiteral' **13:57:49**       .Case(MANGLING_MODULE_OBJC, ForeignModuleKind::Imported) **13:57:49**             ^~~~~~~~~~~~~~~~~~~~ **13:57:49** /home/buildnode/jenkins/workspace/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/llvm/include/llvm/ADT/StringRef.h:857:9: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'const char [4]' to 'const llvm::StringLiteral &' for 1st argument **13:57:49**   class StringLiteral : public StringRef { **13:57:49**         ^ **13:57:49** /home/buildnode/jenkins/workspace/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/llvm/include/llvm/ADT/StringRef.h:857:9: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'const char [4]' to 'llvm::StringLiteral &&' for 1st argument **13:57:49** /home/buildnode/jenkins/workspace/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/llvm/include/llvm/ADT/StringRef.h:864:15: note: candidate disabled: invalid string literal **13:57:49**     constexpr StringLiteral(const char (&Str)[N]) **13:57:49**               ^ **13:57:49** /home/buildnode/jenkins/workspace/oss-swift-4.2-incremental-RA-linux-ubuntu-16_04/llvm/include/llvm/ADT/StringSwitch.h:70:36: note: passing argument to parameter 'S' here **13:57:49**   StringSwitch &Case(StringLiteral S, T Value) { **13:57:49**                                    ^ **13:57:49** 1 warning and 1 error generated.

That was my instinct as well.

Yes. I am fixing now.

1 Like

https://github.com/apple/swift/pull/15985

Just waiting for testing to finish...

Michael