FailedChanges

Summary

  1. [llvm-exegesis] [NFC] Fixing typo. Reviewers: courbet, gchatelet Reviewed By: courbet, gchatelet Subscribers: tschuett, llvm-commits Differential Revision: https://reviews.llvm.org/D54895
  2. Recommit [NFC] Better encapsulation of llvm::Optional Storage Second attempt, trying to navigate out of the UB zone using union for storage instead of raw bytes. I'm prepared to revert that commit as soon as validation breaks, which is likely to happen.
  3. Revert r354244 "[DAGCombiner] Eliminate dead stores to stack." Breaks some bots.
  4. [DAGCombiner] Eliminate dead stores to stack. Summary: A store to an object whose lifetime is about to end can be removed. See PR40550 for motivation. Reviewers: niravd Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D57541
  5. [MC] Make SubtargetFeatureKV only store one FeatureBitset and use an 'unsigned' to hold the value. This class is used for two difference tablegen generated tables. For one of the tables the Value FeatureBitset only has one bit set. For the other usage the Implies field was unused. This patch changes the Value field to just be an unsigned. For the usage that put a real vector in bitset, we now use the previously unused Implies field and leave the Value field unused instead. This is good for a 16K reduction in the size of llc on my local build with all targets enabled.
  6. gn build: Merge r354156
Revision 354250 by gchatelet:
[llvm-exegesis] [NFC] Fixing typo.

Reviewers: courbet, gchatelet

Reviewed By: courbet, gchatelet

Subscribers: tschuett, llvm-commits

Differential Revision: https://reviews.llvm.org/D54895
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/docs/CommandGuide/llvm-exegesis.rst (diff)llvm.src/docs/CommandGuide/llvm-exegesis.rst
Revision 354246 by serge_sans_paille:
Recommit [NFC] Better encapsulation of llvm::Optional Storage

Second attempt, trying to navigate out of the UB zone using
union for storage instead of raw bytes.

I'm prepared to revert that commit as soon as validation breaks,
which is likely to happen.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/ADT/Optional.h (diff)llvm.src/include/llvm/ADT/Optional.h
Revision 354245 by courbet:
Revert r354244 "[DAGCombiner] Eliminate dead stores to stack."

Breaks some bots.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAG.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAG.h
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAGAddressAnalysis.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAGAddressAnalysis.h
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAGNodes.h
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
The file was modified/llvm/trunk/test/CodeGen/BPF/remove_truncate_5.ll (diff)llvm.src/test/CodeGen/BPF/remove_truncate_5.ll
The file was modified/llvm/trunk/test/CodeGen/X86/swap.ll (diff)llvm.src/test/CodeGen/X86/swap.ll
The file was modified/llvm/trunk/test/DebugInfo/COFF/inlining.ll (diff)llvm.src/test/DebugInfo/COFF/inlining.ll
The file was modified/llvm/trunk/test/DebugInfo/COFF/lexicalblock.ll (diff)llvm.src/test/DebugInfo/COFF/lexicalblock.ll
Revision 354244 by courbet:
[DAGCombiner] Eliminate dead stores to stack.

Summary:
A store to an object whose lifetime is about to end can be removed.

See PR40550 for motivation.

Reviewers: niravd

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D57541
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAG.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAG.h
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAGAddressAnalysis.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAGAddressAnalysis.h
The file was modified/llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h (diff)llvm.src/include/llvm/CodeGen/SelectionDAGNodes.h
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGAddressAnalysis.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
The file was modified/llvm/trunk/test/CodeGen/BPF/remove_truncate_5.ll (diff)llvm.src/test/CodeGen/BPF/remove_truncate_5.ll
The file was modified/llvm/trunk/test/CodeGen/X86/swap.ll (diff)llvm.src/test/CodeGen/X86/swap.ll
The file was modified/llvm/trunk/test/DebugInfo/COFF/inlining.ll (diff)llvm.src/test/DebugInfo/COFF/inlining.ll
The file was modified/llvm/trunk/test/DebugInfo/COFF/lexicalblock.ll (diff)llvm.src/test/DebugInfo/COFF/lexicalblock.ll
Revision 354243 by ctopper:
[MC] Make SubtargetFeatureKV only store one FeatureBitset and use an 'unsigned' to hold the value.

This class is used for two difference tablegen generated tables. For one of the tables the Value FeatureBitset only has one bit set. For the other usage the Implies field was unused.

This patch changes the Value field to just be an unsigned. For the usage that put a real vector in bitset, we now use the previously unused Implies field and leave the Value field unused instead.

This is good for a 16K reduction in the size of llc on my local build with all targets enabled.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/MC/SubtargetFeature.h (diff)llvm.src/include/llvm/MC/SubtargetFeature.h
The file was modified/llvm/trunk/lib/MC/SubtargetFeature.cpp (diff)llvm.src/lib/MC/SubtargetFeature.cpp
The file was modified/llvm/trunk/utils/TableGen/SubtargetEmitter.cpp (diff)llvm.src/utils/TableGen/SubtargetEmitter.cpp
Revision 354242 by nico:
gn build: Merge r354156
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/compiler-rt/lib/hwasan/BUILD.gn (diff)llvm.src/utils/gn/secondary/compiler-rt/lib/hwasan/BUILD.gn
The file was modified/llvm/trunk/utils/gn/secondary/compiler-rt/lib/sanitizer_common/BUILD.gn (diff)llvm.src/utils/gn/secondary/compiler-rt/lib/sanitizer_common/BUILD.gn