SuccessChanges

Summary

  1. [docs][Bugpoint] Fix build break. Bugpoint.rst:124: WARNING: Mismatch: both interpreted text role prefix and reference suffix.
  2. [X86] Use APInt::operator<<= and APInt::lshrInPlace. NFC
  3. [SimplifyDemandedBits] Use APInt::intersects to instead of ANDing and comparing to 0 separately. NFC
  4. [docs][Bugpoint]Add notes about multiple crashes Summary: When reducing case for a CodeGenCrash, bugpoint may generate a new reduced testcase that exposes/causes another crash or break something due to limitation. Bugpoint does not distiguish different crashes currently, so when this happens, bugpoint will go on reducing for the new crash, or just abort, we can't get the case reduced for the origial crash. An advice is added into usage doc to connect to recommend checking error message with scripts and `-compile-command`. Reviewers: modocache, bogner, sebpop, reames, vsk, MatzeB Reviewed By: vsk Subscribers: mehdi_amini, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D66832
  5. [X86] Simplify b2b KSHIFTL+KSHIFTR using demanded elts.
  6. [X86] Call SimplifyDemandedVectorElts on KSHIFTL/KSHIFTR nodes during DAG combine.
  7. [X86] Simplify some code in LowerBUILD_VECTORvXi1. NFCI The case were Immediate is 0 and HasConstElts is true should never happen since that would mean the constant elts were all zero. But we check for all zero build vector earlier. So just use HasConstElts and blindly take Immediate without checking if its 0. Move the code that bitcasts and extract the immediate into the the HasConstElts case since the other code just creates an undef with the right type. No casting needed.
  8. Use 'BOOL' instead of BOOL in diagnostic messages Type names should be enclosed in single quotes.
  9. [AMDGPU] Added MI bit IsDOT NFC, needed for future commit. Differential Revision: https://reviews.llvm.org/D67669
  10. gn build: Merge r372149
  11. GSYM: Add the llvm::gsym::Header header class with tests This patch adds the llvm::gsym::Header class which appears at the start of a stand alone GSYM file, or in the first bytes of the GSYM data in a GSYM section within a file. Added encode and decode methods with full error handling and full tests. Differential Revision: https://reviews.llvm.org/D67666
  12. [TableGen] CodeGenMapTable - Don't dereference a dyn_cast result. NFCI. The static analyzer is warning about potential null dereferences of dyn_cast<> results - in these cases we can safely use cast<> directly as we know that these cases should all be the correct type, which is why its working atm and anyway cast<> will assert if they aren't.
Revision 372160 by jsji:
[docs][Bugpoint] Fix build break.

Bugpoint.rst:124: WARNING: Mismatch: both interpreted text role prefix
and reference suffix.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/Bugpoint.rst (diff)llvm.src/docs/Bugpoint.rst
Revision 372159 by ctopper:
[X86] Use APInt::operator<<= and APInt::lshrInPlace. NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
Revision 372158 by ctopper:
[SimplifyDemandedBits] Use APInt::intersects to instead of ANDing and comparing to 0 separately. NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/TargetLowering.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/TargetLowering.cpp
Revision 372157 by jsji:
[docs][Bugpoint]Add notes about multiple crashes

Summary:
    When reducing case for a CodeGenCrash, bugpoint may generate a new
    reduced
    testcase that exposes/causes another crash or break something due to
    limitation.

    Bugpoint does not distiguish different crashes currently,
    so when this happens, bugpoint will go on reducing for the new crash,
    or just abort, we can't get the case reduced for the origial crash.

    An advice is added into usage doc to connect to recommend checking error
    message with scripts and `-compile-command`.

Reviewers: modocache, bogner, sebpop, reames, vsk, MatzeB

Reviewed By: vsk

Subscribers: mehdi_amini, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D66832
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/Bugpoint.rst (diff)llvm.src/docs/Bugpoint.rst
Revision 372155 by ctopper:
[X86] Simplify b2b KSHIFTL+KSHIFTR using demanded elts.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-ext.ll (diff)llvm.src/test/CodeGen/X86/avx512-ext.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-mask-op.ll (diff)llvm.src/test/CodeGen/X86/avx512-mask-op.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512vl-vec-masked-cmp.ll (diff)llvm.src/test/CodeGen/X86/avx512vl-vec-masked-cmp.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vec_smulo.ll (diff)llvm.src/test/CodeGen/X86/vec_smulo.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vec_umulo.ll (diff)llvm.src/test/CodeGen/X86/vec_umulo.ll
Revision 372154 by ctopper:
[X86] Call SimplifyDemandedVectorElts on KSHIFTL/KSHIFTR nodes during DAG combine.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-ext.ll (diff)llvm.src/test/CodeGen/X86/avx512-ext.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-mask-op.ll (diff)llvm.src/test/CodeGen/X86/avx512-mask-op.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vec_smulo.ll (diff)llvm.src/test/CodeGen/X86/vec_smulo.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vec_umulo.ll (diff)llvm.src/test/CodeGen/X86/vec_umulo.ll
Revision 372153 by ctopper:
[X86] Simplify some code in LowerBUILD_VECTORvXi1. NFCI

The case were Immediate is 0 and HasConstElts is true should never
happen since that would mean the constant elts were all zero. But
we check for all zero build vector earlier. So just use HasConstElts
and blindly take Immediate without checking if its 0.

Move the code that bitcasts and extract the immediate into the
the HasConstElts case since the other code just creates an undef
with the right type. No casting needed.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (diff)llvm.src/lib/Target/X86/X86ISelLowering.cpp
Revision 372152 by epilk:
Use 'BOOL' instead of BOOL in diagnostic messages

Type names should be enclosed in single quotes.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (diff)clang.src/include/clang/Basic/DiagnosticSemaKinds.td
The file was modified/cfe/trunk/test/Sema/objc-bool-constant-conversion.m (diff)clang.src/test/Sema/objc-bool-constant-conversion.m
The file was modified/cfe/trunk/test/Sema/tautological-objc-bool-compare.m (diff)clang.src/test/Sema/tautological-objc-bool-compare.m
Revision 372151 by rampitec:
[AMDGPU] Added MI bit IsDOT

NFC, needed for future commit.

Differential Revision: https://reviews.llvm.org/D67669
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIDefines.h (diff)llvm.src/lib/Target/AMDGPU/SIDefines.h
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIInstrFormats.td (diff)llvm.src/lib/Target/AMDGPU/SIInstrFormats.td
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIInstrInfo.h (diff)llvm.src/lib/Target/AMDGPU/SIInstrInfo.h
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIInstrInfo.td (diff)llvm.src/lib/Target/AMDGPU/SIInstrInfo.td
The file was modified/llvm/trunk/lib/Target/AMDGPU/VOP2Instructions.td (diff)llvm.src/lib/Target/AMDGPU/VOP2Instructions.td
The file was modified/llvm/trunk/lib/Target/AMDGPU/VOP3PInstructions.td (diff)llvm.src/lib/Target/AMDGPU/VOP3PInstructions.td
Revision 372150 by gnsyncbot:
gn build: Merge r372149
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/llvm/lib/DebugInfo/GSYM/BUILD.gn (diff)llvm.src/utils/gn/secondary/llvm/lib/DebugInfo/GSYM/BUILD.gn
Revision 372149 by gclayton:
GSYM: Add the llvm::gsym::Header header class with tests

This patch adds the llvm::gsym::Header class which appears at the start of a stand alone GSYM file, or in the first bytes of the GSYM data in a GSYM section within a file. Added encode and decode methods with full error handling and full tests.

Differential Revision: https://reviews.llvm.org/D67666
Change TypePath in RepositoryPath in Workspace
The file was added/llvm/trunk/include/llvm/DebugInfo/GSYM/Header.hllvm.src/include/llvm/DebugInfo/GSYM/Header.h
The file was modified/llvm/trunk/lib/DebugInfo/GSYM/CMakeLists.txt (diff)llvm.src/lib/DebugInfo/GSYM/CMakeLists.txt
The file was added/llvm/trunk/lib/DebugInfo/GSYM/Header.cppllvm.src/lib/DebugInfo/GSYM/Header.cpp
The file was modified/llvm/trunk/unittests/DebugInfo/GSYM/GSYMTest.cpp (diff)llvm.src/unittests/DebugInfo/GSYM/GSYMTest.cpp
Revision 372146 by rksimon:
[TableGen] CodeGenMapTable - Don't dereference a dyn_cast result. NFCI.

The static analyzer is warning about potential null dereferences of dyn_cast<> results - in these cases we can safely use cast<> directly as we know that these cases should all be the correct type, which is why its working atm and anyway cast<> will assert if they aren't.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/TableGen/CodeGenMapTable.cpp (diff)llvm.src/utils/TableGen/CodeGenMapTable.cpp