This is an archive of the discontinued LLVM Phabricator instance.

Differential D80509

buildkite test 2
AbandonedPublic

Authored by goncharov on May 25 2020, 2:33 AM.

Details

Reviewers
None

Diff Detail

Unit TestsFailed

TimeTest
50 msClangd.Clangd::Unknown Unit Message ("")
50 msClangd.Clangd::Unknown Unit Message ("")
130 msClangd.Clangd::Unknown Unit Message ("")

Revision Contents

PathSize
clang-tools-extra/
clangd/
5 lines
test/
2 lines

Diff 265981

clang-tools-extra/clangd/RIFF.cpp

//===--- RIFF.cpp - Binary container file format --------------------------===// //===--- RIFF.cpp - Binary container file format --------------------------===//
Lint: Lint
Inline Actions

clang-format suggested style edits found:

Lint: Lint: clang-format suggested style edits found:
// //
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information. // See https://llvm.org/LICENSE.txt for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
// //
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
#include "RIFF.h" #include "RIFF.h"
Show All 28 Lines
} }
llvm::raw_ostream &operator<<(llvm::raw_ostream &OS, const Chunk &C) { llvm::raw_ostream &operator<<(llvm::raw_ostream &OS, const Chunk &C) {
OS.write(C.ID.data(), C.ID.size()); OS.write(C.ID.data(), C.ID.size());
char Size[4]; char Size[4];
llvm::support::endian::write32le(Size, C.Data.size()); llvm::support::endian::write32le(Size, C.Data.size());
OS.write(Size, sizeof(Size)); OS.write(Size, sizeof(Size));
OS << C.Data; OS << C.Data;
if (C.Data.size() % 2) if (C.Data.size() % 2) OS.write(0);
Lint: Pre-merge checks
Inline Actions

clang-format: please reformat the code

-  if (C.Data.size() % 2) OS.write(0);
+  if (C.Data.size() % 2)
+    OS.write(0);
Lint: Pre-merge checks: clang-format: please reformat the code ``` - if (C.Data.size() % 2) OS.write(0); + if (C.Data.
OS.write(0);
return OS; return OS;
} }
llvm::Expected<File> readFile(llvm::StringRef Stream) { llvm::Expected<File> readFile(llvm::StringRef Stream) {
auto RIFF = readChunk(Stream); auto RIFF = readChunk(Stream);
if (!RIFF) if (!RIFF)
return RIFF.takeError(); return RIFF.takeError();
if (RIFF->ID != fourCC("RIFF")) if (RIFF->ID != fourCC("RIFF"))
Show All 21 Linesllvm::raw_ostream &operator<<(llvm::raw_ostream &OS, const File &F) {
OS.write(Size, sizeof(Size)); OS.write(Size, sizeof(Size));
OS.write(F.Type.data(), F.Type.size()); OS.write(F.Type.data(), F.Type.size());
for (const auto &C : F.Chunks) for (const auto &C : F.Chunks)
OS << C; OS << C;
return OS; return OS;
} }
} // namespace riff } // namespace riff
} // namespace clangd }
Lint: Pre-merge checks
Inline Actions

clang-format: please reformat the code

-} 
+} // namespace clangd
Lint: Pre-merge checks: clang-format: please reformat the code ``` -} +} // namespace clangd ```
Lint: Pre-merge checks
Inline Actions

clang-tidy: warning: namespace 'clangd' not terminated with a closing comment [llvm-namespace-comment]
not useful

Lint: Pre-merge checks: clang-tidy: warning: namespace 'clangd' not terminated with a closing comment [llvm-namespace…
} // namespace clang } // namespace clang

clang-tools-extra/clangd/test/code-action-request.test

Show All 12 Lines "params": {
}, },
"range": { "range": {
"start": { "start": {
"line": 0, "line": 0,
"character": 0 "character": 0
}, },
"end": { "end": {
"line": 0, "line": 0,
"character": 4 "character": 6
} }
}, },
"context": { "context": {
"diagnostics": [] "diagnostics": []
} }
} }
} }
# CHECK: "id": 1, # CHECK: "id": 1,
▲ Show 20 Lines • Show All 41 Lines • Show Last 20 Lines