Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-03 Thread Jim Laskey
Committed. On 3-Jan-07, at 12:52 PM, Reid Spencer wrote: Here's the updated version. I tested this last night while I slept. It passed everything. This version checks both integer and boolean types for size < INT_TYPE_SIZE. Reid. On Wed, 2007-01-03 at 06:17 -0400, Jim Laskey wrote: Waiting t

Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-03 Thread Reid Spencer
Here's the updated version. I tested this last night while I slept. It passed everything. This version checks both integer and boolean types for size < INT_TYPE_SIZE. Reid. On Wed, 2007-01-03 at 06:17 -0400, Jim Laskey wrote: > Waiting till tomorrow (today actually.) > > On 3-Jan-07, at 03:43 AM

Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-03 Thread Jim Laskey
Waiting till tomorrow (today actually.) On 3-Jan-07, at 03:43 AM, Evan Cheng wrote: Please don't apply this. It's incorrectly forcing bool parameter to be zero extended. That's wrong for targets (e.g. ppc32) which have 32- bit boolean type. Reid will post a fixed version tomorrow. Evan On Jan

Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-02 Thread Evan Cheng
Please don't apply this. It's incorrectly forcing bool parameter to be zero extended. That's wrong for targets (e.g. ppc32) which have 32- bit boolean type. Reid will post a fixed version tomorrow. Evan On Jan 2, 2007, at 11:04 PM, Reid Spencer wrote: > On Tue, 2007-01-02 at 22:58 -0800, Reid

Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-02 Thread Reid Spencer
On Tue, 2007-01-02 at 22:58 -0800, Reid Spencer wrote: > To make it even easier, here's a replacement for gcc/llvm-types.cpp Scratch that. Sent the wrong file. Try this one. Sorry. Reid. > > Reid. > > On Tue, 2007-01-02 at 22:54 -0800, Reid Spencer wrote: > > The attached patch is already cont

Re: [llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-02 Thread Reid Spencer
To make it even easier, here's a replacement for gcc/llvm-types.cpp Reid. On Tue, 2007-01-02 at 22:54 -0800, Reid Spencer wrote: > The attached patch is already contained in the previous cumulative PR950 > patch. This contains just today's changes so it can be more easily > applied to the Apple i

[llvm-commits] [llvm-gcc] PR950-zext-bool.patch

2007-01-02 Thread Reid Spencer
The attached patch is already contained in the previous cumulative PR950 patch. This contains just today's changes so it can be more easily applied to the Apple internal SVN server. Reid. Index: gcc/llvm-types.cpp === --- gcc/llvm-typ