FailedChanges

Summary

  1. [Driver] Delete --no-add-needed for RHEL7 or newer This is really not needed. gcc driver doesn't add this option. BTW, since binutils 2.22, --no-copy-dt-needed-entries (new name of --no-add-needed) is the default.
  2. [YAMLIO] Wrap explicit specialization in llvm::yaml to appease build bots
  3. [test] Delete trailing spaces from YAML tests after D65566/r365869
  4. [test] Delete trailing spaces from YAML tests
  5. test/Driver/fsanitize.c: delete -target x86_64-linux-gnu from tests that should always be available Follow-up of D64317/r365587.
  6. [AMDGPU] Fixed comment. NFC.
  7. [mips] Fix JmpLink to texternalsym and tglobaladdr on mcroMIPS R6 There is not match for the `MipsJmpLink texternalsym` and `MipsJmpLink tglobaladdr` patterns for microMIPS R6. As a result LLVM incorrectly selects the `JALRC16` compact 2-byte instruction which takes a target instruction address from a register only and assign `R_MIPS_32` relocation for this instruction. This relocation completely overwrites `JALRC16` and nearby instructions. This patch adds missed matching patterns, selects `BALC` instruction and assign a correct `R_MICROMIPS_PC26_S1` relocation. Differential Revision: https://reviews.llvm.org/D64552
  8. [YAMLIO] Remove trailing spaces when outputting maps llvm::yaml::Output::paddedKey unconditionally outputs spaces, which are superfluous if the value to be dumped is a sequence or map. Change `bool NeedsNewLine` to `StringRef Padding` so that it can be overridden to `\n` if the value is a sequence or map. An empty map/sequence is special. It is printed as `{}` or `[]` without a newline, while a non-empty map/sequence follows a newline. To handle this distinction, add another variable `PaddingBeforeContainer` and does the special handling in endMapping/endSequence. Reviewed By: grimar, jhenderson Differential Revision: https://reviews.llvm.org/D64566
  9. [clangd] Implement typeHierarchy/resolve for subtypes Summary: This allows the client to resolve subtypes one level at a time. For supertypes, this is not necessary, because we eagerly compute supertypes and return all levels. Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D64308
Revision 365877 by maskray:
[Driver] Delete --no-add-needed for RHEL7 or newer

This is really not needed. gcc driver doesn't add this option.
BTW, since binutils 2.22, --no-copy-dt-needed-entries (new name of
--no-add-needed) is the default.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Driver/ToolChains/Linux.cppclang.src/lib/Driver/ToolChains/Linux.cpp
Revision 365876 by maskray:
[YAMLIO] Wrap explicit specialization in llvm::yaml to appease build bots
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/unittests/Support/YAMLIOTest.cppllvm.src/unittests/Support/YAMLIOTest.cpp
Revision 365874 by maskray:
[test] Delete trailing spaces from YAML tests after D65566/r365869
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/test/clang-include-fixer/merge.testclang-tools-extra.src/test/clang-include-fixer/merge.test
The file was modified/clang-tools-extra/trunk/unittests/clang-doc/YAMLGeneratorTest.cppclang-tools-extra.src/unittests/clang-doc/YAMLGeneratorTest.cpp
Revision 365873 by maskray:
[test] Delete trailing spaces from YAML tests
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/unittests/Tooling/DiagnosticsYamlTest.cppclang.src/unittests/Tooling/DiagnosticsYamlTest.cpp
The file was modified/cfe/trunk/unittests/Tooling/RefactoringActionRulesTest.cppclang.src/unittests/Tooling/RefactoringActionRulesTest.cpp
The file was modified/cfe/trunk/unittests/Tooling/RefactoringTest.cppclang.src/unittests/Tooling/RefactoringTest.cpp
The file was modified/cfe/trunk/unittests/Tooling/ReplacementsYamlTest.cppclang.src/unittests/Tooling/ReplacementsYamlTest.cpp
Revision 365872 by maskray:
test/Driver/fsanitize.c: delete -target x86_64-linux-gnu from tests that should always be available

Follow-up of D64317/r365587.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Driver/fsanitize.cclang.src/test/Driver/fsanitize.c
Revision 365871 by rampitec:
[AMDGPU] Fixed comment. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/IR/IntrinsicsAMDGPU.tdllvm.src/include/llvm/IR/IntrinsicsAMDGPU.td
Revision 365870 by atanasyan:
[mips] Fix JmpLink to texternalsym and tglobaladdr on mcroMIPS R6

There is not match for the `MipsJmpLink texternalsym` and `MipsJmpLink
tglobaladdr` patterns for microMIPS R6. As a result LLVM incorrectly
selects the `JALRC16` compact 2-byte instruction which takes a target
instruction address from a register only and assign `R_MIPS_32` relocation
for this instruction. This relocation completely overwrites `JALRC16`
and nearby instructions.

This patch adds missed matching patterns, selects `BALC` instruction and
assign a correct `R_MICROMIPS_PC26_S1` relocation.

Differential Revision: https://reviews.llvm.org/D64552
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.tdllvm.src/lib/Target/Mips/MicroMips32r6InstrInfo.td
The file was modified/llvm/trunk/lib/Target/Mips/MipsScheduleGeneric.tdllvm.src/lib/Target/Mips/MipsScheduleGeneric.td
The file was modified/llvm/trunk/test/CodeGen/Mips/llvm-ir/fptosi.llllvm.src/test/CodeGen/Mips/llvm-ir/fptosi.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/micromips-delay-slot.llllvm.src/test/CodeGen/Mips/micromips-delay-slot.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/micromips-target-external-symbol-reloc.llllvm.src/test/CodeGen/Mips/micromips-target-external-symbol-reloc.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/tailcall/tailcall.llllvm.src/test/CodeGen/Mips/tailcall/tailcall.ll
Revision 365869 by maskray:
[YAMLIO] Remove trailing spaces when outputting maps

llvm::yaml::Output::paddedKey unconditionally outputs spaces, which
are superfluous if the value to be dumped is a sequence or map.
Change `bool NeedsNewLine` to `StringRef Padding` so that it can be
overridden to `\n` if the value is a sequence or map.

An empty map/sequence is special. It is printed as `{}` or `[]` without
a newline, while a non-empty map/sequence follows a newline. To handle
this distinction, add another variable `PaddingBeforeContainer` and does
the special handling in endMapping/endSequence.

Reviewed By: grimar, jhenderson

Differential Revision: https://reviews.llvm.org/D64566
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/YAMLTraits.hllvm.src/include/llvm/Support/YAMLTraits.h
The file was modified/llvm/trunk/lib/Support/YAMLTraits.cppllvm.src/lib/Support/YAMLTraits.cpp
The file was modified/llvm/trunk/unittests/BinaryFormat/MsgPackDocumentTest.cppllvm.src/unittests/BinaryFormat/MsgPackDocumentTest.cpp
The file was modified/llvm/trunk/unittests/Support/YAMLIOTest.cppllvm.src/unittests/Support/YAMLIOTest.cpp
The file was modified/llvm/trunk/unittests/TextAPI/ELFYAMLTest.cppllvm.src/unittests/TextAPI/ELFYAMLTest.cpp
The file was modified/llvm/trunk/unittests/TextAPI/TextStubV1Tests.cppllvm.src/unittests/TextAPI/TextStubV1Tests.cpp
The file was modified/llvm/trunk/unittests/TextAPI/TextStubV2Tests.cppllvm.src/unittests/TextAPI/TextStubV2Tests.cpp
Revision 365867 by nridge:
[clangd] Implement typeHierarchy/resolve for subtypes

Summary:
This allows the client to resolve subtypes one level at a time.

For supertypes, this is not necessary, because we eagerly compute
supertypes and return all levels.

Reviewers: sammccall

Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64308
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clangd/ClangdLSPServer.cppclang-tools-extra.src/clangd/ClangdLSPServer.cpp
The file was modified/clang-tools-extra/trunk/clangd/ClangdLSPServer.hclang-tools-extra.src/clangd/ClangdLSPServer.h
The file was modified/clang-tools-extra/trunk/clangd/ClangdServer.cppclang-tools-extra.src/clangd/ClangdServer.cpp
The file was modified/clang-tools-extra/trunk/clangd/ClangdServer.hclang-tools-extra.src/clangd/ClangdServer.h
The file was modified/clang-tools-extra/trunk/clangd/Protocol.cppclang-tools-extra.src/clangd/Protocol.cpp
The file was modified/clang-tools-extra/trunk/clangd/Protocol.hclang-tools-extra.src/clangd/Protocol.h
The file was modified/clang-tools-extra/trunk/clangd/XRefs.cppclang-tools-extra.src/clangd/XRefs.cpp
The file was modified/clang-tools-extra/trunk/clangd/XRefs.hclang-tools-extra.src/clangd/XRefs.h
The file was modified/clang-tools-extra/trunk/clangd/test/type-hierarchy.testclang-tools-extra.src/clangd/test/type-hierarchy.test
The file was modified/clang-tools-extra/trunk/clangd/unittests/TypeHierarchyTests.cppclang-tools-extra.src/clangd/unittests/TypeHierarchyTests.cpp