[PATCH] D50461: DAG: Don't use ABI copies in some contexts

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 8 09:56:08 PDT 2018


arsenm created this revision.
arsenm added reviewers: sdardis, bogner, nhaehnle.
Herald added subscribers: tpr, wdng.

If an ABI-like value is used in a different block,
the type split used is not necessarily the same as
the call's ABI. The value is used through an intermediate
copy virtual registers from the other block. This
resulted in copies with inconsistent sizes later.

      

Fixes regressions since r338197 when AMDGPU started
splitting vector types for calls.


https://reviews.llvm.org/D50461

Files:
  lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  test/CodeGen/AMDGPU/cross-block-use-is-not-abi-copy.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50461.159748.patch
Type: text/x-patch
Size: 8527 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180808/c3357451/attachment.bin>


More information about the llvm-commits mailing list