[PATCH] D27867: AMDGPU: Add Assert[SZ]Ext during argument load creation

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 10:34:31 PST 2017


arsenm added inline comments.


================
Comment at: test/CodeGen/AMDGPU/min.ll:83-91
 ; SI: v_min_i32
 ; SI: v_min_i32
 ; SI: v_min_i32
 ; SI: v_min_i32
 
-; SI: s_endpgm
+; VI: v_min_i32
+; VI: v_min_i32
----------------
kzhuravl wrote:
> Merge and rename to GCN label?
These should be different since I would hope the mins would be legalized to i16 min, so I think it's better to keep them separate to show this is a problem


https://reviews.llvm.org/D27867





More information about the llvm-commits mailing list