FailedChanges

Summary

  1. [VPlan] Silence static analyzer dyn_cast null dereference warning. NFCI.
  2. AMDGPUPrintfRuntimeBinding - silence static analyzer null dereference warnings. NFCI.
  3. [MIPS] Don't dereference dyn_cast<> Constant results. NFCI. The static analyzer is warning about potential null dereferences, but we should be able to use cast<> directly and if not assert will fire for us.
  4. [Hexagon] Don't dereference dyn_cast<ConstantFPSDNode> result. NFCI. The static analyzer is warning about potential null dereference, but we should be able to use cast<ConstantFPSDNode> directly and if not assert will fire for us.
  5. [Cost][X86] Add v2i64 truncation costs We are missing costs for a lot of truncation cases, I'm hoping to address all the 'zero cost' cases in trunc.ll I thought this was a vector widening side effect, but even before this we had some interesting LV decisions (notably over indvars) being made due to these zero costs.
  6. Clang-format: Add Whitesmiths indentation style Summary: This patch adds support for the Whitesmiths indentation style to clang-format. It’s an update to a patch submitted in 2015 (D6833), but reworks it to use the newer API. There are still some issues with this patch, primarily around `switch` and `case` support. The added unit test won’t currently pass because of the remaining issues. Reviewers: mboehme, MyDeveloperDay, djasper Reviewed By: MyDeveloperDay Subscribers: krasimir, MyDeveloperDay, echristo, cfe-commits Patch By: @timwoj (Tim Wojtulewicz) Tags: #clang Differential Revision: https://reviews.llvm.org/D67627
  7. Move classes into anonymous namespaces. NFC.
  8. [X86] Add test memset and memcpy testcases for D67874. NFC
Revision 372502 by rksimon:
[VPlan] Silence static analyzer dyn_cast null dereference warning. NFCI.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Vectorize/VPlanHCFGTransforms.cpp (diff)llvm.src/lib/Transforms/Vectorize/VPlanHCFGTransforms.cpp
Revision 372501 by rksimon:
AMDGPUPrintfRuntimeBinding - silence static analyzer null dereference warnings. NFCI.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp (diff)llvm.src/lib/Target/AMDGPU/AMDGPUPrintfRuntimeBinding.cpp
Revision 372500 by rksimon:
[MIPS] Don't dereference dyn_cast<> Constant results. NFCI.

The static analyzer is warning about potential null dereferences, but we should be able to use cast<> directly and if not assert will fire for us.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp (diff)llvm.src/lib/Target/Mips/MipsSEISelDAGToDAG.cpp
Revision 372499 by rksimon:
[Hexagon] Don't dereference dyn_cast<ConstantFPSDNode> result. NFCI.

The static analyzer is warning about potential null dereference, but we should be able to use cast<ConstantFPSDNode> directly and if not assert will fire for us.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/Hexagon/HexagonISelDAGToDAG.cpp (diff)llvm.src/lib/Target/Hexagon/HexagonISelDAGToDAG.cpp
Revision 372498 by rksimon:
[Cost][X86] Add v2i64 truncation costs

We are missing costs for a lot of truncation cases, I'm hoping to address all the 'zero cost' cases in trunc.ll

I thought this was a vector widening side effect, but even before this we had some interesting LV decisions (notably over indvars) being made due to these zero costs.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (diff)llvm.src/lib/Target/X86/X86TargetTransformInfo.cpp
The file was modified/llvm/trunk/test/Analysis/CostModel/X86/arith-fix.ll (diff)llvm.src/test/Analysis/CostModel/X86/arith-fix.ll
The file was modified/llvm/trunk/test/Analysis/CostModel/X86/arith-overflow.ll (diff)llvm.src/test/Analysis/CostModel/X86/arith-overflow.ll
The file was modified/llvm/trunk/test/Analysis/CostModel/X86/cast.ll (diff)llvm.src/test/Analysis/CostModel/X86/cast.ll
The file was modified/llvm/trunk/test/Analysis/CostModel/X86/trunc.ll (diff)llvm.src/test/Analysis/CostModel/X86/trunc.ll
The file was modified/llvm/trunk/test/Transforms/SLPVectorizer/X86/arith-fix.ll (diff)llvm.src/test/Transforms/SLPVectorizer/X86/arith-fix.ll
Revision 372497 by paulhoad:
Clang-format: Add Whitesmiths indentation style

Summary:
This patch adds support for the Whitesmiths indentation style to clang-format. It’s an update to a patch submitted in 2015 (D6833), but reworks it to use the newer API.

There are still some issues with this patch, primarily around `switch` and `case` support. The added unit test won’t currently pass because of the remaining issues.

Reviewers: mboehme, MyDeveloperDay, djasper

Reviewed By: MyDeveloperDay

Subscribers: krasimir, MyDeveloperDay, echristo, cfe-commits

Patch By: @timwoj (Tim Wojtulewicz)

Tags: #clang

Differential Revision: https://reviews.llvm.org/D67627
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/docs/ClangFormatStyleOptions.rst (diff)clang.src/docs/ClangFormatStyleOptions.rst
The file was modified/cfe/trunk/include/clang/Format/Format.h (diff)clang.src/include/clang/Format/Format.h
The file was modified/cfe/trunk/lib/Format/ContinuationIndenter.cpp (diff)clang.src/lib/Format/ContinuationIndenter.cpp
The file was modified/cfe/trunk/lib/Format/Format.cpp (diff)clang.src/lib/Format/Format.cpp
The file was modified/cfe/trunk/lib/Format/UnwrappedLineFormatter.cpp (diff)clang.src/lib/Format/UnwrappedLineFormatter.cpp
The file was modified/cfe/trunk/unittests/Format/FormatTest.cpp (diff)clang.src/unittests/Format/FormatTest.cpp
Revision 372495 by d0k:
Move classes into anonymous namespaces. NFC.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/StaticAnalyzer/Checkers/MallocChecker.cpp (diff)clang.src/lib/StaticAnalyzer/Checkers/MallocChecker.cpp
The file was modified/llvm/trunk/lib/Target/Hexagon/HexagonInstrInfo.cpp (diff)llvm.src/lib/Target/Hexagon/HexagonInstrInfo.cpp
The file was modified/llvm/trunk/lib/Target/PowerPC/PPCISelLowering.cpp (diff)llvm.src/lib/Target/PowerPC/PPCISelLowering.cpp
The file was modified/llvm/trunk/lib/Target/PowerPC/PPCInstrInfo.cpp (diff)llvm.src/lib/Target/PowerPC/PPCInstrInfo.cpp
Revision 372494 by ctopper:
[X86] Add test memset and memcpy testcases for D67874. NFC
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/X86/memcpy.ll (diff)llvm.src/test/CodeGen/X86/memcpy.ll
The file was modified/llvm/trunk/test/CodeGen/X86/memset-nonzero.ll (diff)llvm.src/test/CodeGen/X86/memset-nonzero.ll
The file was modified/llvm/trunk/test/CodeGen/X86/memset-zero.ll (diff)llvm.src/test/CodeGen/X86/memset-zero.ll