[PATCH] D17105: AMDGPU: Fix constant bus use check with subregisters

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 14:54:33 PST 2016


arsenm created this revision.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
Herald added a subscriber: arsenm.

If the two operands to an instruction were both
subregisters of the same super register, it would incorrectly
think this counted as the same constant bus use.
    
This fixes the verifier error in fmin_legacy.ll which
was missing -verify-machineinstrs.

http://reviews.llvm.org/D17105

Files:
  lib/Target/AMDGPU/SIInstrInfo.cpp
  test/CodeGen/AMDGPU/fmax_legacy.ll
  test/CodeGen/AMDGPU/fmin_legacy.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17105.47536.patch
Type: text/x-patch
Size: 5137 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160210/41e89722/attachment.bin>


More information about the llvm-commits mailing list