SuccessChanges

Summary

  1. [RegisterBankInfo] Fix 80-col introduced in r293506. NFC.
  2. [GlobalISel] Add missing doxygen keyword for doxygen groups. NFC
  3. AMDGPU: GFX9 GS and HS shaders always have the scratch wave offset in SGPR5 Reviewers: arsenm, nhaehnle Subscribers: kzhuravl, wdng, yaxunl, dstuttard, tpr, t-tye, llvm-commits Differential Revision: https://reviews.llvm.org/D32645
  4. [GISel]:Skip legalizing Intermediate inst(with generic types) During legalization, targets can create Pseudo Instructions with generic types. We shouldn't try to legalize them. Reviewed by Quentin, dsanders https://reviews.llvm.org/D32575
  5. [JumpThreading] When processing compares, explicitly check that the result type is not a vector rather than check for it being an integer. Compares always return a scalar integer or vector of integers. isIntegerTy returns false for vectors, but that's not completely obvious. So using isVectorTy is less confusing.
  6. [JumpThreading] Change a dyn_cast that is already protected by an isa check to a static cast. Combine the with another static cast. NFC Differential Revision: https://reviews.llvm.org/D32874
  7. [GISel]: Add support to translate ConstantVectors Reviewed by Quentin https://reviews.llvm.org/D32814
  8. Fix whitespace before token-paste of an argument. The whitespace should come from the argument name in the macro expansion, rather than from the token passed to the macro (same as it does when not pasting). Added a new test case for the change in behavior to stringize_space.c. FileCheck'ized macro_paste_commaext.c, tweaked the test case, and added a comment; no behavioral change to this test. Differential Revision: https://reviews.llvm.org/D30427
  9. [ConstantRange] Add LLVM_NODISCARD to ConstantRange since a large number of its methods construct new ConstantRange objects.
  10. [Float2Int] Remove return of ConstantRange from seen method. Nothing uses it so it just creates and discards a ConstantRange object for no reason.
Revision 302202 by qcolombet:
[RegisterBankInfo] Fix 80-col introduced in r293506.

NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h (diff)llvm-revision.src/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h
Revision 302201 by qcolombet:
[GlobalISel] Add missing doxygen keyword for doxygen groups.

NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/IRTranslator.h (diff)llvm-revision.src/llvm/trunk/include/llvm/CodeGen/GlobalISel/IRTranslator.h
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h (diff)llvm-revision.src/llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegBankSelect.h (diff)llvm-revision.src/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegBankSelect.h
The file was modified/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h (diff)llvm-revision.src/llvm/trunk/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h
Revision 302200 by mareko:
AMDGPU: GFX9 GS and HS shaders always have the scratch wave offset in SGPR5

Reviewers: arsenm, nhaehnle

Subscribers: kzhuravl, wdng, yaxunl, dstuttard, tpr, t-tye, llvm-commits

Differential Revision: https://reviews.llvm.org/D32645
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp (diff)llvm-revision.src/llvm/trunk/lib/Target/AMDGPU/SIFrameLowering.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp (diff)llvm-revision.src/llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp
The file was modified/llvm/trunk/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp (diff)llvm-revision.src/llvm/trunk/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp
The file was removed/llvm/trunk/test/CodeGen/AMDGPU/local-stack-slot-bug.llllvm-revision.src/llvm/trunk/test/CodeGen/AMDGPU/local-stack-slot-bug.ll
The file was added/llvm/trunk/test/CodeGen/AMDGPU/scratch-simple.llllvm-revision.src/llvm/trunk/test/CodeGen/AMDGPU/scratch-simple.ll
Revision 302199 by aditya_nandakumar:
[GISel]:Skip legalizing Intermediate inst(with generic types)

During legalization, targets can create Pseudo Instructions with
generic types. We shouldn't try to legalize them.

Reviewed by Quentin, dsanders
https://reviews.llvm.org/D32575
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/Legalizer.cpp (diff)llvm-revision.src/llvm/trunk/lib/CodeGen/GlobalISel/Legalizer.cpp
Revision 302198 by ctopper:
[JumpThreading] When processing compares, explicitly check that the result type is not a vector rather than check for it being an integer.

Compares always return a scalar integer or vector of integers. isIntegerTy returns false for vectors, but that's not completely obvious. So using isVectorTy is less confusing.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Scalar/JumpThreading.cpp (diff)llvm-revision.src/llvm/trunk/lib/Transforms/Scalar/JumpThreading.cpp
Revision 302197 by ctopper:
[JumpThreading] Change a dyn_cast that is already protected by an isa check to a static cast. Combine the with another static cast. NFC

Differential Revision: https://reviews.llvm.org/D32874
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Scalar/JumpThreading.cpp (diff)llvm-revision.src/llvm/trunk/lib/Transforms/Scalar/JumpThreading.cpp
Revision 302196 by aditya_nandakumar:
[GISel]: Add support to translate ConstantVectors

Reviewed by Quentin
https://reviews.llvm.org/D32814
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp (diff)llvm-revision.src/llvm/trunk/lib/CodeGen/GlobalISel/IRTranslator.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll (diff)llvm-revision.src/llvm/trunk/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
Revision 302195 by jyknight:
Fix whitespace before token-paste of an argument.

The whitespace should come from the argument name in the macro
expansion, rather than from the token passed to the macro (same as it
does when not pasting).

Added a new test case for the change in behavior to stringize_space.c.

FileCheck'ized macro_paste_commaext.c, tweaked the test case, and
added a comment; no behavioral change to this test.

Differential Revision: https://reviews.llvm.org/D30427
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Lex/TokenLexer.cpp (diff)llvm-revision.src/cfe/trunk/lib/Lex/TokenLexer.cpp
The file was modified/cfe/trunk/test/Preprocessor/macro_paste_commaext.c (diff)llvm-revision.src/cfe/trunk/test/Preprocessor/macro_paste_commaext.c
The file was modified/cfe/trunk/test/Preprocessor/stringize_space.c (diff)llvm-revision.src/cfe/trunk/test/Preprocessor/stringize_space.c
Revision 302194 by ctopper:
[ConstantRange] Add LLVM_NODISCARD to ConstantRange since a large number of its methods construct new ConstantRange objects.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/IR/ConstantRange.h (diff)llvm-revision.src/llvm/trunk/include/llvm/IR/ConstantRange.h
Revision 302193 by ctopper:
[Float2Int] Remove return of ConstantRange from seen method. Nothing uses it so it just creates and discards a ConstantRange object for no reason.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Transforms/Scalar/Float2Int.h (diff)llvm-revision.src/llvm/trunk/include/llvm/Transforms/Scalar/Float2Int.h
The file was modified/llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp (diff)llvm-revision.src/llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp