[PATCH] D28135: SelectionDAG: Fix in legalization of UMAX/SMAX/UMIN/SMIN. Solves PR31486.

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a subscriber: test.
uabelho added a comment.

There were more test cases already prefixed with "r600-" than with "r600." so I chose "r600-". I'll change if "r600." is more correct?

As for FileCheck, sure I can add some checks. The output is

  .text
  .section        .AMDGPU.config
  .long   166100
  .long   2
  .long   165900
  .long   0
  .long   166120
  .long   0
  .text
  .globl  test
  .p2align        8
  .type   test, at function

test:                                   ; @test
; BB#0:                                 ; %bb

  ALU 4, @6, KC0[CB0:0-32], KC1[]
  MEM_RAT_CACHELESS STORE_RAW T0.XY, T1.X, 0
  ALU 3, @11, KC0[], KC1[]
  MEM_RAT_CACHELESS STORE_RAW T0.XY, T1.X, 1
  CF_END
  PAD
  ALU clause starting at 6:
    MOV   T0.X, literal.x,  
    MOV   T0.Y, 0.0,  
    LSHR * T1.X, KC0[2].Y, literal.x,  
  2(2.802597e-45), 0(0.000000e+00)
    MOV * T0.W, KC0[2].Y,  
  ALU clause starting at 11:
    MAX_UINT   T0.X, T0.X, literal.x,  
    MOV   T0.Y, 0.0,  
    LSHR * T1.X, T0.W, literal.y,  
  4(5.605194e-45), 2(2.802597e-45)

.Lfunc_end0:

  .size   test, .Lfunc_end0-test
  
  .section        .AMDGPU.csdata
  ;SQ_PGM_RESOURCES:STACK_SIZE = 0
  
  .section        ".note.GNU-stack"

What CHECKs should I add?


https://reviews.llvm.org/D28135