[all-commits] [llvm/llvm-project] e62531: [DAGCombiner] Remove unnecessary commonAlignment f...

Craig Topper via All-commits all-commits at lists.llvm.org
Tue Feb 13 23:26:20 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e6253102a7c63d73594699d93b1b412f0795ec11
      https://github.com/llvm/llvm-project/commit/e6253102a7c63d73594699d93b1b412f0795ec11
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2024-02-13 (Tue, 13 Feb 2024)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

  Log Message:
  -----------
  [DAGCombiner] Remove unnecessary commonAlignment from CombineExtLoad. (#81705)

The getAlign function for a load returns the commonAlignment of the
"base align" and the offset stored in the MachinePointerInfo.

We're splitting a load here, so we should take the base alignment from
the original load without any offset that may already exist in the
original load. The new load can then maintain its own alignment using
just the base alignment and its own offset.

Noticed by inspection.




More information about the All-commits mailing list