[PATCH] D13085: AMDGPU: Simplify VOP2 operand legalization

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 19:32:05 PDT 2015


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

Checking the legality of src0 is pointless since
it can accept any type of operand.
    
Also skips the more complex checks for checking
literal operands and constant bus restrictions.
These should not be a concern when performing
legalization.
    
This gets called quite a few times and the
attempts at commuting are a significant fraction
of the time spent in SIFixSGPRCopies, so it's
somewhat worthwhile to optimize.


http://reviews.llvm.org/D13085

Files:
  lib/Target/AMDGPU/SIInstrInfo.cpp
  lib/Target/AMDGPU/SIInstrInfo.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13085.35455.patch
Type: text/x-patch
Size: 6632 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150923/e6fa03ae/attachment.bin>


More information about the llvm-commits mailing list