[PATCH] [SDAG] Add SDNodes for umin, umax, smin and smax

[Demikhovsky, Elena]

I agree, I think you can commit it. I'll add target lowering for X86.

-  Elena


-----Original Message-----
From: James Molloy [mailto:james.molloy at arm.com] 
Sent: Thursday, May 14, 2015 22:07
To: james.molloy at arm.com; resistor at mac.com; t.p.northover at gmail.com; Demikhovsky, Elena
Cc: amara.emerson at arm.com; llvm-commits at cs.uiuc.edu
Subject: Re: [PATCH] [SDAG] Add SDNodes for umin, umax, smin and smax

Hi Elena,

Well, the code as it now stands is at least future proofed :) it hasn't added any complexity to support multiple values, so I suggest it just stays like that.

Cheers,

James


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9746

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/


---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.