[PATCH] D77454: LoadInst should store Align, not MaybeAlign.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 22 14:09:39 PDT 2020


efriedma updated this revision to Diff 259385.
efriedma marked 2 inline comments as done.
efriedma retitled this revision from "[WIP] LoadInst should store Align, not MaybeAlign." to "LoadInst should store Align, not MaybeAlign.".
efriedma edited the summary of this revision.
efriedma added a comment.

Rebase.  Address review comments.

This is based on D78403 <https://reviews.llvm.org/D78403>. Regression tests should pass now.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77454/new/

https://reviews.llvm.org/D77454

Files:
  llvm/include/llvm/IR/Instructions.h
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  llvm/lib/IR/Core.cpp
  llvm/lib/IR/Instructions.cpp
  llvm/lib/Target/NVPTX/NVPTXLowerAggrCopies.cpp
  llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
  llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
  llvm/lib/Transforms/InstCombine/InstCombinePHI.cpp
  llvm/lib/Transforms/Scalar/AlignmentFromAssumptions.cpp
  llvm/lib/Transforms/Scalar/GVNHoist.cpp
  llvm/lib/Transforms/Scalar/SROA.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77454.259385.patch
Type: text/x-patch
Size: 17764 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200422/b1d1607b/attachment.bin>


More information about the llvm-commits mailing list