Re: [llvm-commits] [llvm] r47220 - in /llvm/trunk: lib/Transforms/IPO/GlobalOpt.cpp test/Transforms/GlobalOpt/2008-02-16-NestAttr.ll

2008-02-17 Thread Chris Lattner
On Feb 16, 2008, at 6:41 PM, Duncan Sands wrote: > Hi Chris, >>> Remove any 'nest' parameter attributes if the function >>> is not passed as an argument to a trampoline intrinsic. >> >> Nice. Out of curiousity, how does nest do to codegen? > > 'nest' causes a specific register to be grabbed for th

[llvm-commits] [llvm] r47237 - /llvm/trunk/lib/Target/X86/README.txt

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 13:43:57 2008 New Revision: 47237 URL: http://llvm.org/viewvc/llvm-project?rev=47237&view=rev Log: move PR2053 to here. Modified: llvm/trunk/lib/Target/X86/README.txt Modified: llvm/trunk/lib/Target/X86/README.txt URL: http://llvm.org/viewvc/llvm-project/llv

[llvm-commits] [llvm-gcc-4.2] r47235 - /llvm-gcc-4.2/trunk/gcc/config/i386/i386.h

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 13:00:29 2008 New Revision: 47235 URL: http://llvm.org/viewvc/llvm-project?rev=47235&view=rev Log: Fix support for -m32 on x86_64-unknown-linux-gnu. This has the effect of always making the target triple (in llvm bc files) be i386-* instead of i586-* etc, but this

[llvm-commits] [llvm] r47239 - /llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 14:44:51 2008 New Revision: 47239 URL: http://llvm.org/viewvc/llvm-project?rev=47239&view=rev Log: make the logic for breaking up subtracts more explicit, no functionality change. Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp Modified: llvm/trun

[llvm-commits] [llvm] r47240 - /llvm/trunk/test/Transforms/Reassociate/subtest.ll

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 14:48:43 2008 New Revision: 47240 URL: http://llvm.org/viewvc/llvm-project?rev=47240&view=rev Log: upgrade and simplify this test. Modified: llvm/trunk/test/Transforms/Reassociate/subtest.ll Modified: llvm/trunk/test/Transforms/Reassociate/subtest.ll URL: ht

[llvm-commits] [llvm] r47242 - /llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 14:54:40 2008 New Revision: 47242 URL: http://llvm.org/viewvc/llvm-project?rev=47242&view=rev Log: fix pasto Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp URL: http://llvm.org/viewvc/llv

[llvm-commits] [llvm] r47244 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/addnegneg.ll

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 15:03:36 2008 New Revision: 47244 URL: http://llvm.org/viewvc/llvm-project?rev=47244&view=rev Log: Fold (-x + -y) -> -(x+y) which promotes better association, fixing the second half of PR2047 Added: llvm/trunk/test/Transforms/InstCombine/addnegneg.ll Modified:

[llvm-commits] [llvm] r47241 - in /llvm/trunk: lib/Transforms/Scalar/Reassociate.cpp test/Transforms/Reassociate/subtest2.ll

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 14:51:26 2008 New Revision: 47241 URL: http://llvm.org/viewvc/llvm-project?rev=47241&view=rev Log: Split up subtracts into add+negate if they have a reassociable use or operand that is also a subtract. This implements PR2047 and Transforms/Reassociate/subtest2.ll

[llvm-commits] [llvm] r47247 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 15:29:08 2008 New Revision: 47247 URL: http://llvm.org/viewvc/llvm-project?rev=47247&view=rev Log: Teach getModRefInfo that memcpy, memmove, and memset don't "capture" memory addresses. Also, noalias arguments are be considered "like" stack allocated ones for thi

Re: [llvm-commits] [llvm] r47213 - in /llvm/trunk/lib: CodeGen/SelectionDAG/LegalizeDAG.cpp Target/ARM/ARMISelLowering.cpp Target/CBackend/CBackend.cpp Target/CellSPU/SPUISelLowering.cpp Target/IA64/I

2008-02-17 Thread Evan Cheng
Thanks. Evan On Feb 16, 2008, at 6:46 AM, Andrew Lenharth wrote: > Author: alenhar2 > Date: Sat Feb 16 08:46:26 2008 > New Revision: 47213 > > URL: http://llvm.org/viewvc/llvm-project?rev=47213&view=rev > Log: > I cannot find a libgcc function for this builtin. Therefor > expanding it to a noo

Re: [llvm-commits] [llvm] r47086 - in /llvm/trunk: include/llvm/ADT/APInt.h lib/Support/APInt.cpp

2008-02-17 Thread Evan Cheng
Ping. Is this fixed? Does it affect build with -Werror set? Evan On Feb 13, 2008, at 5:22 PM, Bill Wendling wrote: > Hi Dan, > >> +uint32_t APInt::countTrailingOnes() const { >> + if (isSingleWord()) >> +return std::min(uint32_t(CountTrailingOnes_64(VAL)), BitWidth); >> + uint32_t Count =

[llvm-commits] [llvm] r47248 - in /llvm/trunk: include/llvm/Argument.h lib/VMCore/Function.cpp

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 17:22:28 2008 New Revision: 47248 URL: http://llvm.org/viewvc/llvm-project?rev=47248&view=rev Log: Add a predicate to Argument to check for the StructRet attribute. Modified: llvm/trunk/include/llvm/Argument.h llvm/trunk/lib/VMCore/Function.cpp Modified:

[llvm-commits] [llvm] r47249 - /llvm/trunk/include/llvm/Support/CallSite.h

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 18:10:55 2008 New Revision: 47249 URL: http://llvm.org/viewvc/llvm-project?rev=47249&view=rev Log: Add support for setting parameters to CallSite. Modified: llvm/trunk/include/llvm/Support/CallSite.h Modified: llvm/trunk/include/llvm/Support/CallSite.h URL:

[llvm-commits] [llvm] r47250 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 20:11:28 2008 New Revision: 47250 URL: http://llvm.org/viewvc/llvm-project?rev=47250&view=rev Log: bitcasts of pointers are always pointers. If we see a memcpy of a pointer, make sure to check later uses of the pointer as well. Modified: llvm/trunk/lib/Analysi

Re: [llvm-commits] [llvm] r47247 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Chris Lattner
On Feb 17, 2008, at 1:29 PM, Owen Anderson wrote: > Author: resistor > Date: Sun Feb 17 15:29:08 2008 > New Revision: 47247 > > URL: http://llvm.org/viewvc/llvm-project?rev=47247&view=rev > Log: > Teach getModRefInfo that memcpy, memmove, and memset don't "capture" > memory addresses. > Also, n

[llvm-commits] [llvm] r47251 - /llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp

2008-02-17 Thread Chris Lattner
Author: lattner Date: Sun Feb 17 20:18:25 2008 New Revision: 47251 URL: http://llvm.org/viewvc/llvm-project?rev=47251&view=rev Log: simplify some code, BreakUpSubtract always returns nonnull now. Modified: llvm/trunk/lib/Transforms/Scalar/Reassociate.cpp Modified: llvm/trunk/lib/Transforms/S

[llvm-commits] [llvm] r47252 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 20:31:23 2008 New Revision: 47252 URL: http://llvm.org/viewvc/llvm-project?rev=47252&view=rev Log: Fix bugs that Chris noticed in my last patch. Modified: llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp Modified: llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

Re: [llvm-commits] [llvm] r47248 - in /llvm/trunk: include/llvm/Argument.h lib/VMCore/Function.cpp

2008-02-17 Thread Duncan Sands
Hi Owen, > +bool Argument::hasStructRetAttr() const { > + if (!isa(getType())) return false; > + return getParent()->paramHasAttr(getArgNo()+1, ParamAttr::StructRet); > +} since the sret argument (if any) must be the first argument, you could also do a quick return if the argument number is not

[llvm-commits] [llvm] r47254 - /llvm/trunk/lib/VMCore/Function.cpp

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 22:06:26 2008 New Revision: 47254 URL: http://llvm.org/viewvc/llvm-project?rev=47254&view=rev Log: Duncan pointed out that we can fast fail here, because the sret parameter of a function must be the first parameter. Modified: llvm/trunk/lib/VMCore/Function.cp

[llvm-commits] [llvm] r47255 - /llvm/trunk/docs/LangRef.html

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 22:09:01 2008 New Revision: 47255 URL: http://llvm.org/viewvc/llvm-project?rev=47255&view=rev Log: Make the definition of the noalias attribute clearer. Modified: llvm/trunk/docs/LangRef.html Modified: llvm/trunk/docs/LangRef.html URL: http://llvm.org/viewv

[llvm-commits] [llvm] r47256 - /llvm/trunk/docs/LangRef.html

2008-02-17 Thread Duncan Sands
Author: baldrick Date: Sun Feb 17 22:19:38 2008 New Revision: 47256 URL: http://llvm.org/viewvc/llvm-project?rev=47256&view=rev Log: Clarify that 'sret' only applies to pointers, and only applies to the first parameter. Modified: llvm/trunk/docs/LangRef.html Modified: llvm/trunk/docs/LangRef

[llvm-commits] [llvm] r47253 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Owen Anderson
Author: resistor Date: Sun Feb 17 21:52:21 2008 New Revision: 47253 URL: http://llvm.org/viewvc/llvm-project?rev=47253&view=rev Log: Fix a comment, and a bug where we weren't applying the tail call logic in cases that failed the first test. Modified: llvm/trunk/lib/Analysis/BasicAliasAnalysi

Re: [llvm-commits] [llvm] r47247 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Duncan Sands
Hi Owen, > - (isa(Object) && cast(Object)->hasByValAttr( > { > - // Okay, the pointer is to a stack allocated object. If we can prove > that > + (isa(Object) && > + (cast(Object)->hasByValAttr() || > + >

Re: [llvm-commits] [llvm] r47254 - /llvm/trunk/lib/VMCore/Function.cpp

2008-02-17 Thread Chris Lattner
On Feb 17, 2008, at 8:06 PM, Owen Anderson wrote: > Author: resistor > Date: Sun Feb 17 22:06:26 2008 > New Revision: 47254 > > URL: http://llvm.org/viewvc/llvm-project?rev=47254&view=rev > Log: > Duncan pointed out that we can fast fail here, because the sret > parameter of > a function must b

[llvm-commits] [llvm] r47257 - /llvm/trunk/test/Transforms/InstCombine/stacksaverestore.ll

2008-02-17 Thread Chris Lattner
Author: lattner Date: Mon Feb 18 00:11:00 2008 New Revision: 47257 URL: http://llvm.org/viewvc/llvm-project?rev=47257&view=rev Log: upgrade this test. Modified: llvm/trunk/test/Transforms/InstCombine/stacksaverestore.ll Modified: llvm/trunk/test/Transforms/InstCombine/stacksaverestore.ll URL

[llvm-commits] [llvm] r47258 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/stacksaverestore.ll

2008-02-17 Thread Chris Lattner
Author: lattner Date: Mon Feb 18 00:12:38 2008 New Revision: 47258 URL: http://llvm.org/viewvc/llvm-project?rev=47258&view=rev Log: optimize away stackrestore calls that have no intervening alloca or call. Modified: llvm/trunk/lib/Transforms/Scalar/InstructionCombining.cpp llvm/trunk/test

Re: [llvm-commits] [llvm] r47086 - in /llvm/trunk: include/llvm/ADT/APInt.h lib/Support/APInt.cpp

2008-02-17 Thread Bill Wendling
Dan submitted a fix last week. -bw On Feb 17, 2008, at 2:16 PM, Evan Cheng wrote: > Ping. Is this fixed? Does it affect build with -Werror set? > > Evan > > On Feb 13, 2008, at 5:22 PM, Bill Wendling wrote: > >> Hi Dan, >> >>> +uint32_t APInt::countTrailingOnes() const { >>> + if (isSingleWord(

Re: [llvm-commits] [llvm] r47252 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Bill Wendling
What are the bugs? -bw On Feb 17, 2008, at 6:31 PM, Owen Anderson wrote: > Author: resistor > Date: Sun Feb 17 20:31:23 2008 > New Revision: 47252 > > URL: http://llvm.org/viewvc/llvm-project?rev=47252&view=rev > Log: > Fix bugs that Chris noticed in my last patch. > > Modified: > llvm/trunk

Re: [llvm-commits] [llvm] r47252 - /llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp

2008-02-17 Thread Chris Lattner
On Feb 17, 2008, at 10:27 PM, Bill Wendling wrote: > What are the bugs? Specifically, it is good to say what you're fixing in the commit log, so that it shows up in svn log. -Chris > > -bw > > On Feb 17, 2008, at 6:31 PM, Owen Anderson wrote: > >> Author: resistor >> Date: Sun Feb 17 20:31:2

[llvm-commits] [llvm] r47261 - /llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp

2008-02-17 Thread Chris Lattner
Author: lattner Date: Mon Feb 18 01:42:56 2008 New Revision: 47261 URL: http://llvm.org/viewvc/llvm-project?rev=47261&view=rev Log: switch simplifycfg from using vectors for most things to smallvectors, this speeds it up 2.3% on eon. Modified: llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp