SuccessChanges

Summary

  1. Fixing build error from commit 95cbc3d [Attributor] Liveness analysis. Liveness analysis abstract attribute used to indicate which BasicBlocks are dead and can therefore be ignored. Right now we are only looking at noreturn calls. Reviewers: jdoerfert, uenoku Subscribers: hiraditya, llvm-commits Differential Revision: https://reviews.llvm.org/D64162
  2. [Statepoints] Fix a bug in statepoint lowering for functions w/no-realign-stack We were silently using the ABI alignment for all of the stores generated for deopt and gc values. We'd gotten the alignment of the stack slot itself properly reduced (via MachineFrameInfo's clamping), but having the MMO on the store incorrect was enough for us to generate an aligned store to a unaligned location. The simplest fix would have been to just pass the alignment to the helper function, but once we do that, the helper function doesn't really help. So, inline it and directly call the MMO version of DAG.getStore with a properly constructed MMO. Note that there's a separate performance possibility here. Even if we *can* realign stacks, we probably don't *want to* if all of the stores are in slowpaths. But that's a later patch, if at all. :)
  3. Fix pointer width in test from r366754.
  4. gn build: Wrap two comments to 80 columns
  5. [DWARF] Add more error handling to debug line parser. This patch exnteds the error handling in the debug line parser to get rid of the existing MD5 assertion. I want to reuse the debug line parser from LLVM in LLDB where we cannot crash on invalid input. Differential revision: https://reviews.llvm.org/D64544
Revision 366769 by sstefan:
Fixing build error from commit 95cbc3d

[Attributor] Liveness analysis.

Liveness analysis abstract attribute used to indicate which BasicBlocks are dead and can therefore be ignored.
Right now we are only looking at noreturn calls.

Reviewers: jdoerfert, uenoku

Subscribers: hiraditya, llvm-commits

Differential Revision: https://reviews.llvm.org/D64162
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Transforms/IPO/Attributor.h (diff)llvm.src/include/llvm/Transforms/IPO/Attributor.h
The file was modified/llvm/trunk/include/llvm/Transforms/Utils/Local.h (diff)llvm.src/include/llvm/Transforms/Utils/Local.h
The file was modified/llvm/trunk/lib/Transforms/IPO/Attributor.cpp (diff)llvm.src/lib/Transforms/IPO/Attributor.cpp
The file was modified/llvm/trunk/lib/Transforms/Utils/Local.cpp (diff)llvm.src/lib/Transforms/Utils/Local.cpp
The file was added/llvm/trunk/test/Transforms/FunctionAttrs/liveness.llllvm.src/test/Transforms/FunctionAttrs/liveness.ll
Revision 366765 by reames:
[Statepoints] Fix a bug in statepoint lowering for functions w/no-realign-stack

We were silently using the ABI alignment for all of the stores generated for deopt and gc values.  We'd gotten the alignment of the stack slot itself properly reduced (via MachineFrameInfo's clamping), but having the MMO on the store incorrect was enough for us to generate an aligned store to a unaligned location.

The simplest fix would have been to just pass the alignment to the helper function, but once we do that, the helper function doesn't really help.  So, inline it and directly call the MMO version of DAG.getStore with a properly constructed MMO.

Note that there's a separate performance possibility here.  Even if we *can* realign stacks, we probably don't *want to* if all of the stores are in slowpaths.  But that's a later patch, if at all.  :)
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/CodeGen/SelectionDAG/StatepointLowering.cpp (diff)llvm.src/lib/CodeGen/SelectionDAG/StatepointLowering.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/statepoint-no-realign-stack.ll (diff)llvm.src/test/CodeGen/X86/statepoint-no-realign-stack.ll
Revision 366764 by pcc:
Fix pointer width in test from r366754.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/Analysis/ConstantFolding/gep-alias.ll (diff)llvm.src/test/Analysis/ConstantFolding/gep-alias.ll
Revision 366763 by nico:
gn build: Wrap two comments to 80 columns
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/compiler-rt/lib/builtins/BUILD.gn (diff)llvm.src/utils/gn/secondary/compiler-rt/lib/builtins/BUILD.gn
Revision 366762 by Jonas Devlieghere:
[DWARF] Add more error handling to debug line parser.

This patch exnteds the error handling in the debug line parser to get
rid of the existing MD5 assertion. I want to reuse the debug line parser
from LLVM in LLDB where we cannot crash on invalid input.

Differential revision: https://reviews.llvm.org/D64544
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/DebugInfo/DWARF/DWARFDebugLine.cpp (diff)llvm.src/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
The file was modified/llvm/trunk/test/tools/llvm-dwarfdump/X86/Inputs/debug_line_malformed.s (diff)llvm.src/test/tools/llvm-dwarfdump/X86/Inputs/debug_line_malformed.s
The file was modified/llvm/trunk/test/tools/llvm-dwarfdump/X86/debug_line_invalid.test (diff)llvm.src/test/tools/llvm-dwarfdump/X86/debug_line_invalid.test
The file was modified/llvm/trunk/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp (diff)llvm.src/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp