Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Evan Cheng
Ok. I see PPC does exactly the same thing. But this seems like a temporary solution. Could we add some kind of JIT relocation model which are target specific? Or any method that would allow llc to codegen (by specifying some command line option?) in the exact same way as the JIT would? Eva

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Reid Spencer
On Wed, 2006-12-20 at 00:25 -0800, Evan Cheng wrote: > Ok. I see PPC does exactly the same thing. But this seems like a > temporary solution. Could we add some kind of JIT relocation model > which are target specific? Or any method that would allow llc to > codegen (by specifying some command

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Anton Korobeynikov
Hello, Evan. > Ok. I see PPC does exactly the same thing. But this seems like a > temporary solution. Could we add some kind of JIT relocation model > which are target specific? Or any method that would allow llc to > codegen (by specifying some command line option?) in the exact same > wa

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Anton Korobeynikov
Hello, Chris. > PPC64 has a similar problem. It wants to do codegen in PIC mode, but > it doesn't want the JIT to make dyld stubs. As such, it does 'JIT > PIC' mode, which is like PIC but doesn't do the extra load to go > through dyld stubs. Yes, It's quite the same. At codegen level we k

[llvm-commits] [llvm-commit][llvm-gcc]Select powerpc cpu based on gcc arguments, not tuning settings.

2006-12-20 Thread Jim Laskey
Index: gcc/config/rs6000/rs6000.c === --- gcc/config/rs6000/rs6000.c (revision 121533) +++ gcc/config/rs6000/rs6000.c (working copy) @@ -136,6 +136,10 @@ /* Target cpu type */ +/* APPLE LOCAL begin LLVM */ +const char *rs6000_cpu_ta

[llvm-commits] CVS: llvm/lib/AsmParser/llvmAsmParser.y

2006-12-20 Thread Reid Spencer
Changes in directory llvm/lib/AsmParser: llvmAsmParser.y updated: 1.289 -> 1.290 --- Log message: Allow negative constants for unsigned integers and unsigned constants greater than MAX_INT64 for signed integers. This is now valid and is just waiting for the distinction between signed and unsig

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Evan Cheng
I took a look at mingw related code. It seems to me for mingw that you can set relocation type to pic for llc and static for jit. No? Then GVRequiresExtraLoad can just check relocation type? Also GVRequiresExtraLoad now violates 80-col policy and its style is inconsistent with the rest of th

[llvm-commits] CVS: llvm/test/Feature/testconstants.ll

2006-12-20 Thread Reid Spencer
Changes in directory llvm/test/Feature: testconstants.ll updated: 1.7 -> 1.8 --- Log message: Add some tests for negative constants to unsigned and large positive constants to signed. --- Diffs of the changes: (+7 -0) testconstants.ll |7 +++ 1 files changed, 7 insertions(+) Inde

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp X86Subtarget.h X86TargetMachine.cpp

2006-12-20 Thread Chris Lattner
On Dec 20, 2006, at 12:25 AM, Evan Cheng wrote: > Ok. I see PPC does exactly the same thing. But this seems like a > temporary solution. Could we add some kind of JIT relocation model > which are target specific? Or any method that would allow llc to > codegen (by specifying some command line

[llvm-commits] CVS: llvm/include/llvm/Pass.h

2006-12-20 Thread Devang Patel
Changes in directory llvm/include/llvm: Pass.h updated: 1.63 -> 1.64 --- Log message: Enable old pass manager. New pass manager causes CBE failures in llvm-test. --- Diffs of the changes: (+1 -1) Pass.h |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/include/llvm/Pa

[llvm-commits] CVS: llvm/lib/VMCore/Verifier.cpp

2006-12-20 Thread Chris Lattner
Changes in directory llvm/lib/VMCore: Verifier.cpp updated: 1.176 -> 1.177 --- Log message: Revert the previous patch which was incorrect. This unbreaks eon, but rebreaks invoke-1.ll --- Diffs of the changes: (+2 -1) Verifier.cpp |3 ++- 1 files changed, 2 insertions(+), 1 deletion(-)

[llvm-commits] CVS: llvm/lib/Target/X86/X86Subtarget.cpp

2006-12-20 Thread Anton Korobeynikov
Changes in directory llvm/lib/Target/X86: X86Subtarget.cpp updated: 1.44 -> 1.45 --- Log message: Fixed 80 cols & style violation --- Diffs of the changes: (+4 -2) X86Subtarget.cpp |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) Index: llvm/lib/Target/X86/X86Subtarget.cpp

[llvm-commits] CVS: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp

2006-12-20 Thread Jim Laskey
Changes in directory llvm/lib/Target/PowerPC: PPCAsmPrinter.cpp updated: 1.223 -> 1.224 --- Log message: Changes to target powerpc for non-Darwin assemblers. 1. Patches from Nick Lewycky. 2. Code to filter register names and print them as numeric values on non-Darwin systems. --- Diffs of th

[llvm-commits] CVS: llvm/lib/VMCore/Verifier.cpp

2006-12-20 Thread Chris Lattner
Changes in directory llvm/lib/VMCore: Verifier.cpp updated: 1.177 -> 1.178 --- Log message: Fix Regression/Verifier/invoke-1.ll --- Diffs of the changes: (+7 -2) Verifier.cpp |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) Index: llvm/lib/VMCore/Verifier.cpp diff -u ll

[llvm-commits] CVS: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp

2006-12-20 Thread Jim Laskey
Changes in directory llvm/lib/Target/PowerPC: PPCAsmPrinter.cpp updated: 1.224 -> 1.225 --- Log message: Original patch was overly complicated. --- Diffs of the changes: (+11 -34) PPCAsmPrinter.cpp | 45 +++-- 1 files changed, 11 insertions(+), 34

[llvm-commits] CVS: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp

2006-12-20 Thread Jim Laskey
Changes in directory llvm/lib/Target/PowerPC: PPCAsmPrinter.cpp updated: 1.225 -> 1.226 --- Log message: Oops. --- Diffs of the changes: (+1 -1) PPCAsmPrinter.cpp |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp diff -u llvm/lib/T

[llvm-commits] [llvm-gcc] CastFixes To Make llvm-convert.cpp nearly signless (Review & Commit)

2006-12-20 Thread Reid Spencer
All, The attached patch removes all but one isSigned() calls from llvm-convert.cpp and otherwise makes it signless. This cleans up the casting patch committed yesterday and bases signedness only on TYPE_UNSIGNED(TREE_TYPE(tree)). The one exception can't be removed until the signed and unsigned int

[llvm-commits] CVS: llvm/test/Regression/CodeGen/PowerPC/2006-04-05-splat-ish.ll Frames-align.ll Frames-alloca.ll Frames-large.ll Frames-small.ll fsqrt.ll seteq-0.ll stfiwx.ll

2006-12-20 Thread Jim Laskey
Changes in directory llvm/test/Regression/CodeGen/PowerPC: 2006-04-05-splat-ish.ll updated: 1.2 -> 1.3 Frames-align.ll updated: 1.2 -> 1.3 Frames-alloca.ll updated: 1.2 -> 1.3 Frames-large.ll updated: 1.3 -> 1.4 Frames-small.ll updated: 1.3 -> 1.4 fsqrt.ll updated: 1.3 -> 1.4 seteq-0.ll updated:

[llvm-commits] CVS: llvm/lib/VMCore/PassManager.cpp

2006-12-20 Thread Devang Patel
Changes in directory llvm/lib/VMCore: PassManager.cpp updated: 1.109 -> 1.110 --- Log message: At the beginning of run, initialize analyis info availability for _ALL_ pass managers. Otherwise, stale available analysis info, from the managers not yet run, may cause pass manager to take wrong tur

[llvm-commits] [llvm-gcc] Removal of signedness from gcc/config/i386/i386.h

2006-12-20 Thread Reid Spencer
The attached patch removes signedness from the i386.h file, in particular the LLVM_TARGET_INTRINSIC_LOWER macro. The macro takes three additional arguments and depends on a change in the function its called from, like this: Index: gcc/llvm-convert.cpp ==

Re: [llvm-commits] [llvm-gcc] Removal of signedness from gcc/config/i386/i386.h

2006-12-20 Thread Reid Spencer
(this email actually has the patch attached) The attached patch removes signedness from the i386.h file, in particular the LLVM_TARGET_INTRINSIC_LOWER macro. The macro takes three additional arguments and depends on a change in the function its called from, like this: Index: gcc/llvm-convert.cpp

[llvm-commits] [llvm-gcc] Signedness patch for rs6000.h

2006-12-20 Thread Reid Spencer
Attached is a patch for the gcc/config/rs6000/rs6000.h file to remove isSigned() calls. This is analgous to the last patch I sent but for PPC instead of X86. Same comments apply. This patch has not been compiled or tested as I don't have a PPC machine. Please review and try it. Reid. Index: gcc/

[llvm-commits] ppc linux patch 3

2006-12-20 Thread Nick Lewycky
This is the last patch, adding the LinuxAsmPrinter and LinuxTargetAsmInfo. Most of the code is a copy of the DarwinAsmPrinter. Nick Lewycky Index: lib/Target/PowerPC/PPCAsmPrinter.cpp === RCS file: /var/cvs/llvm/llvm/lib/Target/PowerP

[llvm-commits] ppc linux patch 3.1

2006-12-20 Thread Nick Lewycky
The original patch had a merge error and wouldn't compile. This one has been fixed. As well, a large #if 0 has been replaced with a simple "TODO" comment. Nick Index: lib/Target/PowerPC/PPCAsmPrinter.cpp === RCS file: /var/cvs/llvm/ll

Re: [llvm-commits] [llvm-gcc] CastFixes To Make llvm-convert.cpp nearly signless (Review & Commit)

2006-12-20 Thread Chris Lattner
On Dec 20, 2006, at 3:13 PM, Reid Spencer wrote: > The attached patch removes all but one isSigned() calls from > llvm-convert.cpp and otherwise makes it signless. This cleans up the > casting patch committed yesterday and bases signedness only on > TYPE_UNSIGNED(TREE_TYPE(tree)). The one exception

Re: [llvm-commits] [llvm-gcc] Signedness patch for rs6000.h

2006-12-20 Thread Chris Lattner
On Dec 20, 2006, at 4:46 PM, Reid Spencer wrote: > Attached is a patch for the gcc/config/rs6000/rs6000.h file to remove > isSigned() calls. This is analgous to the last patch I sent but for > PPC > instead of X86. Same comments > apply. > > This patch has not been compiled or tested as I don'

Re: [llvm-commits] [llvm-gcc] Removal of signedness from gcc/config/i386/i386.h

2006-12-20 Thread Chris Lattner
Is there a change to i386.h that goes along with this? -Chris On Dec 20, 2006, at 4:34 PM, Reid Spencer wrote: > (this email actually has the patch attached) > > The attached patch removes signedness from the i386.h file, in > particular the LLVM_TARGET_INTRINSIC_LOWER macro. The macro takes >

Re: [llvm-commits] [llvm-gcc] Signedness patch for rs6000.h

2006-12-20 Thread Reid Spencer
Jim, Don't commit that! These patches are intertwined. After everything is reviewed, I'll submit the whole thing as one patch. If you commit this, it will break on i386. Reid. On Wed, 2006-12-20 at 17:43 -0800, Chris Lattner wrote: > On Dec 20, 2006, at 4:46 PM, Reid Spencer wrote: > > > Atta

Re: [llvm-commits] [llvm-gcc] Removal of signedness from gcc/config/i386/i386.h

2006-12-20 Thread Reid Spencer
Okay, one more time .. On Wed, 2006-12-20 at 16:34 -0800, Reid Spencer wrote: > (this email actually has the patch attached) > > The attached patch removes signedness from the i386.h file, in > particular the LLVM_TARGET_INTRINSIC_LOWER macro. The macro takes three > additional arguments and depe

Re: [llvm-commits] [llvm-gcc] Removal of signedness from gcc/config/i386/i386.h

2006-12-20 Thread Chris Lattner
>> Although this patch removes isSigned() calls and is correct, it is >> not >> optimal. It is still using CastInst::getCastOpcode to figure out the >> casting opcode for it. Perhaps someone with a little more >> knowledge of >> the intrinsic functions could narrow these down to the appropriate

Re: [llvm-commits] SETCC InstructionCombining.cpp [#1/n]

2006-12-20 Thread Chris Lattner
On Dec 17, 2006, at 3:35 PM, Reid Spencer wrote: > Chris, > > Attached is the patch to InstructionCombining.cpp for SETCC conversion > to ICmpInst. This passes all tests. > > All your previous feedback has been incorporated and confirmed. The > test > just completed includes all those changes a

[llvm-commits] CVS: llvm/lib/Analysis/ScalarEvolution.cpp

2006-12-20 Thread Reid Spencer
Changes in directory llvm/lib/Analysis: ScalarEvolution.cpp updated: 1.74 -> 1.75 --- Log message: Add some comments about things that can go away once signless types are in. --- Diffs of the changes: (+8 -1) ScalarEvolution.cpp |9 - 1 files changed, 8 insertions(+), 1 deletio

[llvm-commits] CVS: llvm-test/Makefile.rules

2006-12-20 Thread Reid Spencer
Changes in directory llvm-test: Makefile.rules updated: 1.20 -> 1.21 --- Log message: Restore definitions of LCC1 and LCC1PLUS to cause test programs to be dependent upon the llvm-gcc compiler components they use. --- Diffs of the changes: (+3 -0) Makefile.rules |3 +++ 1 files changed

Re: [llvm-commits] [llvm-gcc] Cast Fix

2006-12-20 Thread Reid Spencer
Chris, You asked for a case where I need to (recursively) determine the sign of a some Value. I have a case in mind: lib/Transforms/Utils/LowerSwitch.cpp. When it decides to lower this to a branch tree it is recursively calling switchConvert and passing the switch value in. The signedness is nee

[llvm-commits] CVS: llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp

2006-12-20 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/IPO: SimplifyLibCalls.cpp updated: 1.78 -> 1.79 --- Log message: Get rid of a useless if statement whose then and else blocks were identical. --- Diffs of the changes: (+1 -4) SimplifyLibCalls.cpp |5 + 1 files changed, 1 insertion(+), 4 del

[llvm-commits] CVS: llvm/lib/Transforms/Instrumentation/ProfilingUtils.cpp

2006-12-20 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Instrumentation: ProfilingUtils.cpp updated: 1.11 -> 1.12 --- Log message: Remove isSigned calls via foreknowledge of main's argument types. --- Diffs of the changes: (+4 -7) ProfilingUtils.cpp | 11 --- 1 files changed, 4 insertions(+), 7