[PATCH] [SDAG] Add SDNodes for umin, umax, smin and smax
Demikhovsky, Elena
elena.demikhovsky at intel.com
Thu May 14 12:26:41 PDT 2015
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.
More information about the llvm-commits
mailing list