FailedChanges

Summary

  1. Reapply r359906, "RegAllocFast: Add heuristic to detect values not live-out of a block" This reverts commit r359912. This should pass now, since the clang test was made less fragile in r359918.
  2. [CommandLine] Enable Grouping for short options by default. Part 4 of 5 Summary: This change enables `cl::Grouping` for short options -- options with names of a single character. This is consistent with GNU getopt behavior. Reviewers: rnk, MaskRay Reviewed By: MaskRay Subscribers: thopre, cfe-commits, MaskRay, rupprecht, hiraditya, llvm-commits Tags: #llvm, #clang Differential Revision: https://reviews.llvm.org/D61270
  3. [DAGCombine] Remove repeated variables. NFCI.
  4. Revert r359906, "RegAllocFast: Add heuristic to detect values not live-out of a block" Makes clang/test/Misc/backend-stack-frame-diagnostics-fallback.cpp fail.
  5. [TargetLowering] SimplifySetCC - remove repeated variable. NFCI. Also reduce scope of Temp variable.
  6. [CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5 Summary: By default, `parseCommandLineOptions()` will accept either a `-` or `--` prefix for long options -- options with names longer than a single character. While this change does not affect behavior, it will be helpful with a subsequent change that requires long options use the `--` prefix. Reviewers: rnk, thopre Reviewed By: thopre Subscribers: thopre, cfe-commits, hiraditya, llvm-commits Tags: #llvm, #clang Differential Revision: https://reviews.llvm.org/D61269
Revision 359919 by arsenm:
Reapply r359906, "RegAllocFast: Add heuristic to detect values not live-out of a block"

This reverts commit r359912.

This should pass now, since the clang test was made less fragile in
r359918.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/RegAllocFast.cpptrunk/lib/CodeGen/RegAllocFast.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/swifterror.lltrunk/test/CodeGen/AArch64/swifterror.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/indirect-addressing-term.lltrunk/test/CodeGen/AMDGPU/indirect-addressing-term.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomic.lltrunk/test/CodeGen/Mips/atomic.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomic64.lltrunk/test/CodeGen/Mips/atomic64.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomicCmpSwapPW.lltrunk/test/CodeGen/Mips/atomicCmpSwapPW.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/dsp-spill-reload.lltrunk/test/CodeGen/Mips/dsp-spill-reload.ll
The file was modified/llvm/trunk/test/CodeGen/PowerPC/addegluecrash.lltrunk/test/CodeGen/PowerPC/addegluecrash.ll
The file was modified/llvm/trunk/test/CodeGen/PowerPC/vsx.lltrunk/test/CodeGen/PowerPC/vsx.ll
The file was modified/llvm/trunk/test/CodeGen/Thumb2/high-reg-spill.mirtrunk/test/CodeGen/Thumb2/high-reg-spill.mir
The file was modified/llvm/trunk/test/CodeGen/X86/atomic-unordered.lltrunk/test/CodeGen/X86/atomic-unordered.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic32.lltrunk/test/CodeGen/X86/atomic32.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic64.lltrunk/test/CodeGen/X86/atomic64.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic6432.lltrunk/test/CodeGen/X86/atomic6432.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-mask-zext-bugfix.lltrunk/test/CodeGen/X86/avx512-mask-zext-bugfix.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr11415.lltrunk/test/CodeGen/X86/pr11415.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr30430.lltrunk/test/CodeGen/X86/pr30430.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr32284.lltrunk/test/CodeGen/X86/pr32284.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr32484.lltrunk/test/CodeGen/X86/pr32484.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr34592.lltrunk/test/CodeGen/X86/pr34592.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr34653.lltrunk/test/CodeGen/X86/pr34653.ll
The file was modified/llvm/trunk/test/CodeGen/X86/swifterror.lltrunk/test/CodeGen/X86/swifterror.ll
The file was modified/llvm/trunk/test/DebugInfo/X86/pieces-1.lltrunk/test/DebugInfo/X86/pieces-1.ll
Revision 359917 by dhinton:
[CommandLine] Enable Grouping for short options by default.  Part 4 of 5

Summary:
This change enables `cl::Grouping` for short options --
options with names of a single character.  This is consistent with GNU
getopt behavior.

Reviewers: rnk, MaskRay

Reviewed By: MaskRay

Subscribers: thopre, cfe-commits, MaskRay, rupprecht, hiraditya, llvm-commits

Tags: #llvm, #clang

Differential Revision: https://reviews.llvm.org/D61270
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/CommandLine.htrunk/include/llvm/Support/CommandLine.h
The file was modified/llvm/trunk/lib/Support/CommandLine.cpptrunk/lib/Support/CommandLine.cpp
The file was modified/llvm/trunk/test/tools/llvm-readobj/merged.testtrunk/test/tools/llvm-readobj/merged.test
Revision 359915 by rksimon:
[DAGCombine] Remove repeated variables. NFCI.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpptrunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Revision 359912 by nico:
Revert r359906, "RegAllocFast: Add heuristic to detect values not live-out of a block"

Makes clang/test/Misc/backend-stack-frame-diagnostics-fallback.cpp fail.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/RegAllocFast.cpptrunk/lib/CodeGen/RegAllocFast.cpp
The file was modified/llvm/trunk/test/CodeGen/AArch64/swifterror.lltrunk/test/CodeGen/AArch64/swifterror.ll
The file was modified/llvm/trunk/test/CodeGen/AMDGPU/indirect-addressing-term.lltrunk/test/CodeGen/AMDGPU/indirect-addressing-term.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomic.lltrunk/test/CodeGen/Mips/atomic.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomic64.lltrunk/test/CodeGen/Mips/atomic64.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/atomicCmpSwapPW.lltrunk/test/CodeGen/Mips/atomicCmpSwapPW.ll
The file was modified/llvm/trunk/test/CodeGen/Mips/dsp-spill-reload.lltrunk/test/CodeGen/Mips/dsp-spill-reload.ll
The file was modified/llvm/trunk/test/CodeGen/PowerPC/addegluecrash.lltrunk/test/CodeGen/PowerPC/addegluecrash.ll
The file was modified/llvm/trunk/test/CodeGen/PowerPC/vsx.lltrunk/test/CodeGen/PowerPC/vsx.ll
The file was modified/llvm/trunk/test/CodeGen/Thumb2/high-reg-spill.mirtrunk/test/CodeGen/Thumb2/high-reg-spill.mir
The file was modified/llvm/trunk/test/CodeGen/X86/atomic-unordered.lltrunk/test/CodeGen/X86/atomic-unordered.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic32.lltrunk/test/CodeGen/X86/atomic32.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic64.lltrunk/test/CodeGen/X86/atomic64.ll
The file was modified/llvm/trunk/test/CodeGen/X86/atomic6432.lltrunk/test/CodeGen/X86/atomic6432.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-mask-zext-bugfix.lltrunk/test/CodeGen/X86/avx512-mask-zext-bugfix.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr11415.lltrunk/test/CodeGen/X86/pr11415.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr30430.lltrunk/test/CodeGen/X86/pr30430.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr32284.lltrunk/test/CodeGen/X86/pr32284.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr32484.lltrunk/test/CodeGen/X86/pr32484.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr34592.lltrunk/test/CodeGen/X86/pr34592.ll
The file was modified/llvm/trunk/test/CodeGen/X86/pr34653.lltrunk/test/CodeGen/X86/pr34653.ll
The file was modified/llvm/trunk/test/CodeGen/X86/swifterror.lltrunk/test/CodeGen/X86/swifterror.ll
The file was modified/llvm/trunk/test/DebugInfo/X86/pieces-1.lltrunk/test/DebugInfo/X86/pieces-1.ll
Revision 359911 by rksimon:
[TargetLowering] SimplifySetCC - remove repeated variable. NFCI.

Also reduce scope of Temp variable.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/TargetLowering.cpptrunk/lib/CodeGen/SelectionDAG/TargetLowering.cpp
Revision 359909 by dhinton:
[CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5

Summary:
By default, `parseCommandLineOptions()` will accept either a
`-` or `--` prefix for long options -- options with names longer than
a single character.

While this change does not affect behavior, it will be helpful with a
subsequent change that requires long options use the `--` prefix.

Reviewers: rnk, thopre

Reviewed By: thopre

Subscribers: thopre, cfe-commits, hiraditya, llvm-commits

Tags: #llvm, #clang

Differential Revision: https://reviews.llvm.org/D61269
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Driver/clang-offload-bundler.cN/A
The file was modified/llvm/trunk/lib/Support/CommandLine.cpptrunk/lib/Support/CommandLine.cpp
The file was modified/llvm/trunk/test/FileCheck/dump-input-enable.txttrunk/test/FileCheck/dump-input-enable.txt
The file was modified/llvm/trunk/test/Support/check-default-options.txttrunk/test/Support/check-default-options.txt
The file was modified/llvm/trunk/unittests/Support/CommandLineTest.cpptrunk/unittests/Support/CommandLineTest.cpp

Summary

  1. Ensure there is stack usage in stack size warning test r359906 broke this because the only stack usage was from a spill which can be avoided since the only block is a return.
  2. Attempt to fix the clang-sphinx-docs bot after r358797
  3. Remove else-after-return
  4. [CUDA][Clang][Bugfix] Add missing CUDA 9.2 case Summary: The bug was reported on the OpenMP-dev list: .../obj-release/lib/clang/9.0.0/include/__clang_cuda_intrinsics.h:173:35: error: '__nvvm_shfl_sync_idx_i32' needs target feature ptx60|ptx61|ptx63|ptx64 __MAKE_SYNC_SHUFFLES(__shfl_sync, __nvvm_shfl_sync_idx_i32, This problem occurs when trying to compile a .cu file that requires a newer ptx version (>ptx60 in this case) than ptx42. Reviewers: tra, ABataev, caomhin Reviewed By: tra Subscribers: jdoerfert, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D61474
  5. [CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5 Summary: By default, `parseCommandLineOptions()` will accept either a `-` or `--` prefix for long options -- options with names longer than a single character. While this change does not affect behavior, it will be helpful with a subsequent change that requires long options use the `--` prefix. Reviewers: rnk, thopre Reviewed By: thopre Subscribers: thopre, cfe-commits, hiraditya, llvm-commits Tags: #llvm, #clang Differential Revision: https://reviews.llvm.org/D61269
Revision 359918 by arsenm:
Ensure there is stack usage in stack size warning test

r359906 broke this because the only stack usage was from a spill which
can be avoided since the only block is a return.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpptrunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp
Revision 359916 by nico:
Attempt to fix the clang-sphinx-docs bot after r358797
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/docs/analyzer/checkers.rsttrunk/docs/analyzer/checkers.rst
Revision 359913 by dblaikie:
Remove else-after-return
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/AST/ExprConstant.cpptrunk/lib/AST/ExprConstant.cpp
Revision 359910 by gbercea:
[CUDA][Clang][Bugfix] Add missing CUDA 9.2 case

Summary:
The bug was reported on the OpenMP-dev list:

.../obj-release/lib/clang/9.0.0/include/__clang_cuda_intrinsics.h:173:35: error: '__nvvm_shfl_sync_idx_i32' needs target feature ptx60|ptx61|ptx63|ptx64
__MAKE_SYNC_SHUFFLES(__shfl_sync, __nvvm_shfl_sync_idx_i32,

This problem occurs when trying to compile a .cu file that requires a newer ptx version (>ptx60 in this case) than ptx42.



Reviewers: tra, ABataev, caomhin

Reviewed By: tra

Subscribers: jdoerfert, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D61474
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Driver/ToolChains/Cuda.cpptrunk/lib/Driver/ToolChains/Cuda.cpp
Revision 359909 by dhinton:
[CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5

Summary:
By default, `parseCommandLineOptions()` will accept either a
`-` or `--` prefix for long options -- options with names longer than
a single character.

While this change does not affect behavior, it will be helpful with a
subsequent change that requires long options use the `--` prefix.

Reviewers: rnk, thopre

Reviewed By: thopre

Subscribers: thopre, cfe-commits, hiraditya, llvm-commits

Tags: #llvm, #clang

Differential Revision: https://reviews.llvm.org/D61269
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Driver/clang-offload-bundler.ctrunk/test/Driver/clang-offload-bundler.c
The file was modified/llvm/trunk/lib/Support/CommandLine.cppN/A
The file was modified/llvm/trunk/test/FileCheck/dump-input-enable.txtN/A
The file was modified/llvm/trunk/test/Support/check-default-options.txtN/A
The file was modified/llvm/trunk/unittests/Support/CommandLineTest.cppN/A

Summary

  1. [hwasan] Fix HWASAN_WITH_INTERCEPTORS=OFF build on not-android. Summary: I'm not aware of any platforms where this will work, but the code should at least compile. HWASAN_WITH_INTERCEPTORS=OFF means there is magic in libc that would call __hwasan_thread_enter / __hwasan_thread_exit as appropriate. Reviewers: pcc, winksaville Subscribers: srhines, kubamracek, #sanitizers, llvm-commits Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D61337
Revision 359914 by eugenis:
[hwasan] Fix HWASAN_WITH_INTERCEPTORS=OFF build on not-android.

Summary:
I'm not aware of any platforms where this will work, but the code should at least compile.
HWASAN_WITH_INTERCEPTORS=OFF means there is magic in libc that would call __hwasan_thread_enter /
__hwasan_thread_exit as appropriate.

Reviewers: pcc, winksaville

Subscribers: srhines, kubamracek, #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D61337
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/lib/hwasan/hwasan_linux.cpptrunk/lib/hwasan/hwasan_linux.cpp