Changes

Summary

Import sys in repo.py. The find function in repo.py calls sys.exit on error. Without this import that call to exit will fail, masking the actual error message. This patch fixes that.
DAGCombine: Combine shuffles of splat-shuffles Summary: Reapply r299047, but this time handle correctly splat-masks with undef elements. Reviewers: spatel, RKSimon, eli.friedman, andreadb Reviewed By: spatel Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D31961
[AArch64] Consider widening instructions in cost calculations The AArch64 instruction set has a few "widening" instructions (e.g., uaddl, saddl, uaddw, etc.) that take one or more doubleword operands and produce quadword results. The operands are automatically sign- or zero-extended as appropriate. However, in LLVM IR, these extends are explicit. This patch updates TTI to consider these widening instructions as single operations whose cost is attached to the arithmetic instruction. It marks extends that are part of a widening operation "free" and applies a sub-target specified overhead (zero by default) to the arithmetic instructions. Differential Revision: https://reviews.llvm.org/D32706
[InstCombine] clean up matchDeMorgansLaws(); NFCI The motivation for getting rid of dyn_castNotVal is to allow fixing: https://bugs.llvm.org/show_bug.cgi?id=32706 So this was supposed to be functional-change-intended for the case of inverting constants and applying DeMorgan. However, I can't find any cases where that pattern will actually get to matchDeMorgansLaws() because we have other folds in visitAnd/visitOr that do the same thing. So this ends up just being a clean-up patch with slight efficiency improvement, but no-functional-change-intended.
clang-format: [JS] Don't indent JavaScript IIFEs. Because IIFEs[1] are often used like an anonymous namespace around large sections of JavaScript code, it's useful not to indent to them (which effectively reduces the column limit by the indent amount needlessly). It's also common for developers to wrap these around entire files or libraries. When adopting clang-format, changing the indent entire file can reduce the usefulness of the blame annotations. Patch by danbeam, thanks! Differential Revision: https://reviews.llvm.org/D32989
[NewGVN] Simplify a DEBUG() statement. NFCI.

Change Type	Path in Repository	Path in Workspace
Revision 302584 by lhames: Import sys in repo.py. The find function in repo.py calls sys.exit on error. Without this import that call to exit will fail, masking the actual error message. This patch fixes that.
	/lldb/trunk/scripts/Xcode/repo.py (diff)	llvm-revision.src/lldb/trunk/scripts/Xcode/repo.py
Revision 302583 by zvi: DAGCombine: Combine shuffles of splat-shuffles Summary: Reapply r299047, but this time handle correctly splat-masks with undef elements. Reviewers: spatel, RKSimon, eli.friedman, andreadb Reviewed By: spatel Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D31961
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (diff)	llvm-revision.src/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
	/llvm/trunk/test/CodeGen/X86/shuffle-of-splat-multiuses.ll (diff)	llvm-revision.src/llvm/trunk/test/CodeGen/X86/shuffle-of-splat-multiuses.ll
Revision 302582 by mssimpso: [AArch64] Consider widening instructions in cost calculations The AArch64 instruction set has a few "widening" instructions (e.g., uaddl, saddl, uaddw, etc.) that take one or more doubleword operands and produce quadword results. The operands are automatically sign- or zero-extended as appropriate. However, in LLVM IR, these extends are explicit. This patch updates TTI to consider these widening instructions as single operations whose cost is attached to the arithmetic instruction. It marks extends that are part of a widening operation "free" and applies a sub-target specified overhead (zero by default) to the arithmetic instructions. Differential Revision: https://reviews.llvm.org/D32706
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h (diff)	llvm-revision.src/llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h
	/llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp (diff)	llvm-revision.src/llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
	/llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.h (diff)	llvm-revision.src/llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.h
	/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (diff)	llvm-revision.src/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
	/llvm/trunk/test/Analysis/CostModel/AArch64/free-widening-casts.ll	llvm-revision.src/llvm/trunk/test/Analysis/CostModel/AArch64/free-widening-casts.ll
Revision 302581 by spatel: [InstCombine] clean up matchDeMorgansLaws(); NFCI The motivation for getting rid of dyn_castNotVal is to allow fixing: https://bugs.llvm.org/show_bug.cgi?id=32706 So this was supposed to be functional-change-intended for the case of inverting constants and applying DeMorgan. However, I can't find any cases where that pattern will actually get to matchDeMorgansLaws() because we have other folds in visitAnd/visitOr that do the same thing. So this ends up just being a clean-up patch with slight efficiency improvement, but no-functional-change-intended.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp (diff)	llvm-revision.src/llvm/trunk/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
Revision 302580 by mprobst: clang-format: [JS] Don't indent JavaScript IIFEs. Because IIFEs[1] are often used like an anonymous namespace around large sections of JavaScript code, it's useful not to indent to them (which effectively reduces the column limit by the indent amount needlessly). It's also common for developers to wrap these around entire files or libraries. When adopting clang-format, changing the indent entire file can reduce the usefulness of the blame annotations. Patch by danbeam, thanks! Differential Revision: https://reviews.llvm.org/D32989
Change Type	Path in Repository	Path in Workspace
	/cfe/trunk/lib/Format/UnwrappedLineParser.cpp (diff)	llvm-revision.src/cfe/trunk/lib/Format/UnwrappedLineParser.cpp
	/cfe/trunk/unittests/Format/FormatTestJS.cpp (diff)	llvm-revision.src/cfe/trunk/unittests/Format/FormatTestJS.cpp
Revision 302579 by davide: [NewGVN] Simplify a DEBUG() statement. NFCI.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (diff)	llvm-revision.src/llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp