FailedChanges

Summary

  1. The variable "Latch" is only used in an assert, which makes builds that use "-DNDEBUG" fail with unused variable messages. Summary: Move the logic into the assert itself. Subscribers: hiraditya, sanjoy, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D64654
  2. [SystemZ] Fix build bot failure after r365932 Insert LLVM_FALLTHROUGH to avoid compiler warning.
  3. [x86] add test for bogus cmov (PR40483); NFC
  4. [AMDGPU] Extend MIMG opcode to 8 bits This is NFC, but required for future commit. Differential Revision: https://reviews.llvm.org/D64649
  5. Addition to rL365925, removing remaining virtuals
  6. [clang-doc] Add html links to references <a> tags are added for the parents and members of records and return type and params of functions. The link redirects to the reference's info file. The directory path where each info file will be saved is now generated in the serialization phase and stored as an attribute in each Info. Bitcode writer and reader were modified to handle the new attributes. Committed on behalf of Diego Astiazarán (diegoaat97@gmail.com). Differential Revision: https://reviews.llvm.org/D63663
  7. Slightly simplify MappedBlockStream::createIndexedStream() calls All callers had a PDBFile object at hand, so call Pdb.createIndexedStream() instead, which pre-populates all the arguments (and returns nullptr for kInvalidStreamIndex). Also change safelyCreateIndexedStream() to only take the string index, and update callers. Make the method public and call it in two places that manually did the bounds checking before. No intended behavior change. Differential Revision: https://reviews.llvm.org/D64633
  8. [WebAssembly] Make pthread imply bulk-memory, mutable-globals Summary: This paves the way for using passive segments in pthread builds, which will make separate memory files unnecessary. Mutable globals are also necessary for the upcoming implementation of TLS. Reviewers: aheejin, dschuff Subscribers: sbc100, jgravelle-google, sunfish, jfb, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D64586
  9. Add explicit newline at end of `llvm-pdbutil dump` All dump modes I checked didn't print a trailing newline, so add one.
Revision 365943 by saugustine:
The variable "Latch" is only used in an assert, which makes builds that use "-DNDEBUG" fail with unused variable messages.

Summary: Move the logic into the assert itself.

Subscribers: hiraditya, sanjoy, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D64654
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Scalar/IndVarSimplify.cpp (diff)llvm.src/lib/Transforms/Scalar/IndVarSimplify.cpp
Revision 365942 by uweigand:
[SystemZ] Fix build bot failure after r365932

Insert LLVM_FALLTHROUGH to avoid compiler warning.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/SystemZ/SystemZInstrInfo.cpp (diff)llvm.src/lib/Target/SystemZ/SystemZInstrInfo.cpp
Revision 365941 by spatel:
[x86] add test for bogus cmov (PR40483); NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/X86/combine-sbb.ll (diff)llvm.src/test/CodeGen/X86/combine-sbb.ll
Revision 365940 by rampitec:
[AMDGPU] Extend MIMG opcode to 8 bits

This is NFC, but required for future commit.

Differential Revision: https://reviews.llvm.org/D64649
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/MIMGInstructions.td (diff)llvm.src/lib/Target/AMDGPU/MIMGInstructions.td
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIInstrFormats.td (diff)llvm.src/lib/Target/AMDGPU/SIInstrFormats.td
Revision 365938 by sstefan:
Addition to rL365925, removing remaining virtuals
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/IPO/Attributor.cpp (diff)llvm.src/lib/Transforms/IPO/Attributor.cpp
Revision 365937 by juliehockett:
[clang-doc] Add html links to references

<a> tags are added for the parents and members of records and return type and
params of functions. The link redirects to the reference's info file.

The directory path where each info file will be saved is now generated in the
serialization phase and stored as an attribute in each Info.

Bitcode writer and reader were modified to handle the new attributes.

Committed on behalf of Diego Astiazarán (diegoaat97@gmail.com).

Differential Revision: https://reviews.llvm.org/D63663
Change TypePath in RepositoryPath in Workspace
The file was modified/clang-tools-extra/trunk/clang-doc/BitcodeReader.cpp (diff)clang-tools-extra.src/clang-doc/BitcodeReader.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/BitcodeWriter.cpp (diff)clang-tools-extra.src/clang-doc/BitcodeWriter.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/BitcodeWriter.h (diff)clang-tools-extra.src/clang-doc/BitcodeWriter.h
The file was modified/clang-tools-extra/trunk/clang-doc/Generators.cpp (diff)clang-tools-extra.src/clang-doc/Generators.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/Generators.h (diff)clang-tools-extra.src/clang-doc/Generators.h
The file was modified/clang-tools-extra/trunk/clang-doc/HTMLGenerator.cpp (diff)clang-tools-extra.src/clang-doc/HTMLGenerator.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/MDGenerator.cpp (diff)clang-tools-extra.src/clang-doc/MDGenerator.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/Representation.cpp (diff)clang-tools-extra.src/clang-doc/Representation.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/Representation.h (diff)clang-tools-extra.src/clang-doc/Representation.h
The file was modified/clang-tools-extra/trunk/clang-doc/Serialize.cpp (diff)clang-tools-extra.src/clang-doc/Serialize.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/YAMLGenerator.cpp (diff)clang-tools-extra.src/clang-doc/YAMLGenerator.cpp
The file was modified/clang-tools-extra/trunk/clang-doc/tool/ClangDocMain.cpp (diff)clang-tools-extra.src/clang-doc/tool/ClangDocMain.cpp
The file was modified/clang-tools-extra/trunk/unittests/clang-doc/HTMLGeneratorTest.cpp (diff)clang-tools-extra.src/unittests/clang-doc/HTMLGeneratorTest.cpp
The file was modified/clang-tools-extra/trunk/unittests/clang-doc/YAMLGeneratorTest.cpp (diff)clang-tools-extra.src/unittests/clang-doc/YAMLGeneratorTest.cpp
Revision 365936 by nico:
Slightly simplify MappedBlockStream::createIndexedStream() calls

All callers had a PDBFile object at hand, so call
Pdb.createIndexedStream() instead, which pre-populates all the arguments
(and returns nullptr for kInvalidStreamIndex).

Also change safelyCreateIndexedStream() to only take the string index,
and update callers. Make the method public and call it in two places
that manually did the bounds checking before.

No intended behavior change.

Differential Revision: https://reviews.llvm.org/D64633
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/DebugInfo/PDB/Native/PDBFile.h (diff)llvm.src/include/llvm/DebugInfo/PDB/Native/PDBFile.h
The file was modified/llvm/trunk/lib/DebugInfo/PDB/Native/DbiStream.cpp (diff)llvm.src/lib/DebugInfo/PDB/Native/DbiStream.cpp
The file was modified/llvm/trunk/lib/DebugInfo/PDB/Native/PDBFile.cpp (diff)llvm.src/lib/DebugInfo/PDB/Native/PDBFile.cpp
The file was modified/llvm/trunk/lib/DebugInfo/PDB/Native/TpiStream.cpp (diff)llvm.src/lib/DebugInfo/PDB/Native/TpiStream.cpp
The file was modified/llvm/trunk/tools/llvm-pdbutil/BytesOutputStyle.cpp (diff)llvm.src/tools/llvm-pdbutil/BytesOutputStyle.cpp
The file was modified/llvm/trunk/tools/llvm-pdbutil/LinePrinter.cpp (diff)llvm.src/tools/llvm-pdbutil/LinePrinter.cpp
The file was modified/llvm/trunk/tools/llvm-pdbutil/YAMLOutputStyle.cpp (diff)llvm.src/tools/llvm-pdbutil/YAMLOutputStyle.cpp
The file was modified/llvm/trunk/tools/llvm-pdbutil/llvm-pdbutil.cpp (diff)llvm.src/tools/llvm-pdbutil/llvm-pdbutil.cpp
Revision 365935 by tlively:
[WebAssembly] Make pthread imply bulk-memory, mutable-globals

Summary:
This paves the way for using passive segments in pthread builds, which
will make separate memory files unnecessary. Mutable globals are also
necessary for the upcoming implementation of TLS.

Reviewers: aheejin, dschuff

Subscribers: sbc100, jgravelle-google, sunfish, jfb, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D64586
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Driver/ToolChains/WebAssembly.cpp (diff)clang.src/lib/Driver/ToolChains/WebAssembly.cpp
The file was modified/cfe/trunk/test/Driver/wasm-toolchain.c (diff)clang.src/test/Driver/wasm-toolchain.c
Revision 365934 by nico:
Add explicit newline at end of `llvm-pdbutil dump`

All dump modes I checked didn't print a trailing newline, so add one.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/tools/llvm-pdbutil/DumpOutputStyle.cpp (diff)llvm.src/tools/llvm-pdbutil/DumpOutputStyle.cpp