Re: [llvm-commits] [llvm] r45022 - in /llvm/trunk: include/llvm/Target/TargetInstrInfo.h lib/Target/Target.td utils/TableGen/CodeGenInstruction.h utils/TableGen/CodeGenTarget.cpp utils/TableGen/InstrI

2007-12-16 Thread Bill Wendling
On Dec 15, 2007, at 1:23 AM, Christopher Lamb wrote: On Dec 14, 2007, at 10:40 AM, Bill Wendling wrote: My understanding of a side effect is when an instruction has un-modeled effects. For instance, "emms" on X86 has side effects that aren't modeled in the instruction pattern -- it resets the F

[llvm-commits] [llvm] r45071 - /llvm/trunk/lib/AsmParser/LLLexer.cpp

2007-12-16 Thread Bill Wendling
Author: void Date: Sun Dec 16 03:16:12 2007 New Revision: 45071 URL: http://llvm.org/viewvc/llvm-project?rev=45071&view=rev Log: Remove spurious warnings from GCC: warning: suggest a space before ';' or explicit braces around empty body in 'for' statement Patch by Mike Stump (modified slightly

[llvm-commits] [llvm-gcc-4.2] r45072 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

2007-12-16 Thread Duncan Sands
Author: baldrick Date: Sun Dec 16 09:39:40 2007 New Revision: 45072 URL: http://llvm.org/viewvc/llvm-project?rev=45072&view=rev Log: Give calls to inline assembler the nounwind attribute. Modified: llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp Modified: llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp URL:

[llvm-commits] [llvm] r45073 - in /llvm/trunk: include/llvm/Support/CallSite.h lib/Transforms/Scalar/InstructionCombining.cpp lib/Transforms/Utils/InlineFunction.cpp lib/VMCore/Instructions.cpp test/C

2007-12-16 Thread Duncan Sands
Author: baldrick Date: Sun Dec 16 09:51:49 2007 New Revision: 45073 URL: http://llvm.org/viewvc/llvm-project?rev=45073&view=rev Log: Make instcombine promote inline asm calls to 'nounwind' calls. Remove special casing of inline asm from the inliner. There is a potential problem: the verifier rej

Re: [llvm-commits] [llvm] r45022 - in /llvm/trunk: include/llvm/Target/TargetInstrInfo.h lib/Target/Target.td utils/TableGen/CodeGenInstruction.h utils/TableGen/CodeGenTarget.cpp utils/TableGen/InstrI

2007-12-16 Thread Chris Lattner
On Dec 13, 2007, at 5:48 PM, Bill Wendling wrote: > URL: http://llvm.org/viewvc/llvm-project?rev=45022&view=rev > Log: > Add flags to indicate that there are "never" side effects or that > there "may be" > side effects for machine instructions. Hi Bill, > +++ llvm/trunk/include/llvm/Target/Targ

[llvm-commits] [llvm] r45074 - in /llvm/trunk/lib/Target/X86: X86InstrFormats.td X86InstrSSE.td

2007-12-16 Thread Chris Lattner
Author: lattner Date: Sun Dec 16 14:12:41 2007 New Revision: 45074 URL: http://llvm.org/viewvc/llvm-project?rev=45074&view=rev Log: Fix the JIT encoding of cmp*ss, which aborts with this assertion currently: X86CodeEmitter.cpp:378: failed assertion `0 && "Immediate size not set!"' I *think* this

[llvm-commits] [llvm] r45075 - /llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

2007-12-16 Thread Chris Lattner
Author: lattner Date: Sun Dec 16 14:26:54 2007 New Revision: 45075 URL: http://llvm.org/viewvc/llvm-project?rev=45075&view=rev Log: fix a questionable cast, thanks to Mike Stump for pointing this out. Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Modified: llvm/trunk/lib/Target/X86

[llvm-commits] [llvm] r45076 - /llvm/trunk/lib/Target/CellSPU/SPUOperands.td

2007-12-16 Thread Chris Lattner
Author: lattner Date: Sun Dec 16 14:41:33 2007 New Revision: 45076 URL: http://llvm.org/viewvc/llvm-project?rev=45076&view=rev Log: don't violate C TBAA rules, use FloatToBits instead. Modified: llvm/trunk/lib/Target/CellSPU/SPUOperands.td Modified: llvm/trunk/lib/Target/CellSPU/SPUOperands.

Re: [llvm-commits] [llvm] r45073 - in /llvm/trunk: include/llvm/Support/CallSite.h lib/Transforms/Scalar/InstructionCombining.cpp lib/Transforms/Utils/InlineFunction.cpp lib/VMCore/Instructions.cpp te

2007-12-16 Thread Chris Lattner
> URL: http://llvm.org/viewvc/llvm-project?rev=45073&view=rev > Log: > Make instcombine promote inline asm calls to 'nounwind' > calls. Nice. > Remove special casing of inline asm from the > inliner. There is a potential problem: the verifier > rejects invokes of inline asm (not sure why). Ple

[llvm-commits] [llvm] r45077 - /llvm/trunk/lib/Transforms/Utils/InlineFunction.cpp

2007-12-16 Thread Duncan Sands
Author: baldrick Date: Sun Dec 16 15:01:21 2007 New Revision: 45077 URL: http://llvm.org/viewvc/llvm-project?rev=45077&view=rev Log: Revert this part of r45073 until the verifier is changed not to reject invoke of inline asm. Modified: llvm/trunk/lib/Transforms/Utils/InlineFunction.cpp Modif

Re: [llvm-commits] [llvm] r45073 - in /llvm/trunk: include/llvm/Support/CallSite.h lib/Transforms/Scalar/InstructionCombining.cpp lib/Transforms/Utils/InlineFunction.cpp lib/VMCore/Instructions.cpp te

2007-12-16 Thread Duncan Sands
Hi Chris, > > Remove special casing of inline asm from the > > inliner. There is a potential problem: the verifier > > rejects invokes of inline asm (not sure why). > > Please restore the inliner to not do this or change the verifier to > not reject it. Right now you have broken the tree. I

Re: [llvm-commits] [llvm] r45073 - in /llvm/trunk: include/llvm/Support/CallSite.h lib/Transforms/Scalar/InstructionCombining.cpp lib/Transforms/Utils/InlineFunction.cpp lib/VMCore/Instructions.cpp te

2007-12-16 Thread Chris Lattner
On Dec 16, 2007, at 1:04 PM, Duncan Sands wrote: > Hi Chris, > >>> Remove special casing of inline asm from the >>> inliner. There is a potential problem: the verifier >>> rejects invokes of inline asm (not sure why). >> >> Please restore the inliner to not do this or change the verifier to >> n

Re: [llvm-commits] [llvm-gcc-4.2] r45054 - /llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c

2007-12-16 Thread Chris Lattner
On Dec 14, 2007, at 5:30 PM, Dale Johannesen wrote: > Author: johannes > Date: Fri Dec 14 19:30:32 2007 > New Revision: 45054 > > URL: http://llvm.org/viewvc/llvm-project?rev=45054&view=rev > Log: > Suppress a valid warning for llvm. Having instcombine emit this warning seems like a really bad t

Re: [llvm-commits] [llvm-gcc-4.2] r45054 - /llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c

2007-12-16 Thread Dale Johannesen
On Dec 16, 2007, at 1:21 PM, Chris Lattner wrote: > > On Dec 14, 2007, at 5:30 PM, Dale Johannesen wrote: > >> Author: johannes >> Date: Fri Dec 14 19:30:32 2007 >> New Revision: 45054 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=45054&view=rev >> Log: >> Suppress a valid warning for llvm.

Re: [llvm-commits] [llvm-gcc-4.2] r45054 - /llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c

2007-12-16 Thread Chris Lattner
On Dec 16, 2007, at 1:29 PM, Dale Johannesen wrote: >>> URL: http://llvm.org/viewvc/llvm-project?rev=45054&view=rev >>> Log: >>> Suppress a valid warning for llvm. >> >> Having instcombine emit this warning seems like a really bad thing to >> do. Should we just stop the optimizer from emitting w

Re: [llvm-commits] [llvm-gcc-4.2] r45054 - /llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c

2007-12-16 Thread Dale Johannesen
On Dec 16, 2007, at 1:32 PM, Chris Lattner wrote: > > On Dec 16, 2007, at 1:29 PM, Dale Johannesen wrote: > URL: http://llvm.org/viewvc/llvm-project?rev=45054&view=rev Log: Suppress a valid warning for llvm. >>> >>> Having instcombine emit this warning seems like a really bad >>>

Re: [llvm-commits] [llvm-gcc-4.2] r45054 - /llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c

2007-12-16 Thread Chris Lattner
On Dec 16, 2007, at 1:59 PM, Dale Johannesen wrote: >> I definitely agree, which is why I added it in the first place :). >> The problem is that it isn't hooked up with any of the other >> diagnostics machinery, so it isn't controllable and doesn't give >> decent location info (for example). >> >

[llvm-commits] [llvm] r45080 - in /llvm/trunk: docs/LangRef.html lib/VMCore/Verifier.cpp

2007-12-16 Thread Christopher Lamb
Author: clamb Date: Sun Dec 16 19:00:21 2007 New Revision: 45080 URL: http://llvm.org/viewvc/llvm-project?rev=45080&view=rev Log: Make it clear in the LangRef that allocation instructions only operated on the generic address space. Implement support in the verifier for ensuring this is true. Mo

[llvm-commits] [llvm-gcc-4.0] r45083 - in /llvm-gcc-4.0/trunk/gcc: config/i386/llvm-i386.cpp config/rs6000/llvm-rs6000.cpp llvm-abi.h llvm-backend.cpp llvm-convert.cpp llvm-types.cpp

2007-12-16 Thread Christopher Lamb
Author: clamb Date: Sun Dec 16 19:15:41 2007 New Revision: 45083 URL: http://llvm.org/viewvc/llvm-project?rev=45083&view=rev Log: Update to use new PointerType::getUnqual() api. Modified: llvm-gcc-4.0/trunk/gcc/config/i386/llvm-i386.cpp llvm-gcc-4.0/trunk/gcc/config/rs6000/llvm-rs6000.cpp

[llvm-commits] [llvm-gcc-4.2] r45084 - in /llvm-gcc-4.2/trunk/gcc: config/i386/llvm-i386.cpp config/rs6000/llvm-rs6000.cpp llvm-abi.h llvm-backend.cpp llvm-convert.cpp llvm-types.cpp

2007-12-16 Thread Christopher Lamb
Author: clamb Date: Sun Dec 16 19:16:41 2007 New Revision: 45084 URL: http://llvm.org/viewvc/llvm-project?rev=45084&view=rev Log: Update to use new PointerType::getUnqual() api. Modified: llvm-gcc-4.2/trunk/gcc/config/i386/llvm-i386.cpp llvm-gcc-4.2/trunk/gcc/config/rs6000/llvm-rs6000.cpp

Re: [llvm-commits] [llvm] r45082 - in /llvm/trunk: examples/BrainF/ include/llvm/ lib/AsmParser/ lib/Bitcode/Reader/ lib/CodeGen/ lib/ExecutionEngine/ lib/Target/MSIL/ lib/Transforms/IPO/ lib/Transfor

2007-12-16 Thread Chris Lattner
Nice. Please email llvmdev about this API change to help reduce the number of people that are surprised. Thanks Christopher! -Chris http://nondot.org/sabre http://llvm.org On Dec 16, 2007, at 5:12 PM, Christopher Lamb <[EMAIL PROTECTED] > wrote: > Author: clamb > Date: Sun Dec 16 19:12:55

[llvm-commits] [llvm-gcc-4.0] r45088 - /llvm-gcc-4.0/trunk/gcc/llvm-convert.cpp

2007-12-16 Thread Duncan Sands
Author: baldrick Date: Mon Dec 17 00:34:34 2007 New Revision: 45088 URL: http://llvm.org/viewvc/llvm-project?rev=45088&view=rev Log: Add the nounwind attribute to inline asm calls. Modified: llvm-gcc-4.0/trunk/gcc/llvm-convert.cpp Modified: llvm-gcc-4.0/trunk/gcc/llvm-convert.cpp URL: http: