[llvm-commits] [PATCH] Missing setcc enumerations

[Eli Friedman]

On Tue, Dec 13, 2011 at 6:29 PM, Villmow, Micah <Micah.Villmow at amd.com> wrote:
> I don't have commit access yet, maybe I should request it.

Applied in r146548.  Also, see http://llvm.org/docs/DeveloperPolicy.html.

-Eli

>
>> -----Original Message-----
>> From: Eli Friedman [mailto:eli.friedman at gmail.com]
>> Sent: Tuesday, December 13, 2011 5:07 PM
>> To: Villmow, Micah
>> Cc: llvm-commits
>> Subject: Re: [llvm-commits] [PATCH] Missing setcc enumerations
>>
>> On Tue, Dec 13, 2011 at 12:31 PM, Villmow, Micah <Micah.Villmow at amd.com>
>> wrote:
>> > This patch adds in a few setcc enumerations that are missing. A test
>> case
>> > that hits this with an internal version of the AMDIL backend is
>> attached.
>> >
>> >
>> >
>> > The issue is caused by DAGCombiner merging the A op B | B op A into a
>> single
>> > setcc, where it ends up being SETTRUE.
>>
>> Patch looks fine.
>>
>> -Eli
>
>