[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-05-03 Thread Chris Lattner
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.45 -> 1.46 --- Log message: remove extraneous type qualifiers --- Diffs of the changes: (+4 -4) MathExtras.h |8 1 files changed, 4 insertions(+), 4 deletions(-) Index: llvm/include/llvm/Support/MathExtr

Re: [llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-24 Thread Chris Lattner
On Mar 22, 2007, at 1:18 PM, Reid Spencer wrote: > On Thu, 2007-03-22 at 12:35 -0700, Jeff Cohen wrote: >> How is it any worse than checking for GCC? > > I don't like that either :) FWIW, I don't see anything wrong with Jeff's change. MathExtras may be suboptimal w.r.t to libsystem, but so is

Re: [llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-22 Thread Reid Spencer
On Thu, 2007-03-22 at 12:35 -0700, Jeff Cohen wrote: > How is it any worse than checking for GCC? I don't like that either :) > > It's treated as an intrinsic when optimizations are enable. It directly > generates a bswap instruction. Same as the conditional code for GCC. Okay, sounds like i

Re: [llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-22 Thread Jeff Cohen
Reid Spencer wrote: > >> -for (uint64_t Shift = 64 >> 1; Shift; Shift >>= 1) { >> +for (unsigned Shift = 64 >> 1; Shift; Shift >>= 1) { >> > > Why not uint32_t? For conformity with the rest of your changes? The size of a shift amount is unrelated to the size of the function's argumen

Re: [llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-22 Thread Jeff Cohen
How is it any worse than checking for GCC? It's treated as an intrinsic when optimizations are enable. It directly generates a bswap instruction. Same as the conditional code for GCC. Reid Spencer wrote: > Jeff, > > On Thu, 2007-03-22 at 14:12 -0500, Jeff Cohen wrote: > >> Changes in direct

Re: [llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-22 Thread Reid Spencer
Jeff, On Thu, 2007-03-22 at 14:12 -0500, Jeff Cohen wrote: > > Changes in directory llvm/include/llvm/Support: > > MathExtras.h updated: 1.43 -> 1.44 > --- > Log message: > > Be more explicit concerning argument sizes. > Use VC++ byteswap intrinsics. ... snip ... > @@ -93,22 +93,30 @@ > >

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-22 Thread Jeff Cohen
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.43 -> 1.44 --- Log message: Be more explicit concerning argument sizes. Use VC++ byteswap intrinsics. --- Diffs of the changes: (+41 -31) MathExtras.h | 72 +--

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-04 Thread Jeff Cohen
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.42 -> 1.43 --- Log message: Unbreak VC++ build. --- Diffs of the changes: (+1 -1) MathExtras.h |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/include/llvm/Support/MathExtras.h diff -u llvm/inclu

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-04 Thread Chris Lattner
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.41 -> 1.42 --- Log message: fix 80 col violations, mark arrays static --- Diffs of the changes: (+12 -8) MathExtras.h | 20 1 files changed, 12 insertions(+), 8 deletions(-) Index: llvm/include

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-02 Thread Anton Korobeynikov
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.40 -> 1.41 --- Log message: Fix uninitialized use of variable. Remove tabs and fix identation. --- Diffs of the changes: (+19 -19) MathExtras.h | 38 +++--- 1 files changed, 19 inserti

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-01 Thread Owen Anderson
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.39 -> 1.40 --- Log message: Fix a typo. --- Diffs of the changes: (+1 -1) MathExtras.h |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/include/llvm/Support/MathExtras.h diff -u llvm/include/llv

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-03-01 Thread Owen Anderson
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.38 -> 1.39 --- Log message: Use GCC intrinsics when available, and use smarter fallbacks when not. --- Diffs of the changes: (+26 -2) MathExtras.h | 28 ++-- 1 files changed, 26 insertions(+),

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2007-02-28 Thread Owen Anderson
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.37 -> 1.38 --- Log message: Use the GCC built-in for PopulationCount when it's available, and use a faster algorithm when it's not. This should be particularly noticeable in the 64-bit case. --- Diffs of the changes: (+

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-12-18 Thread Chris Lattner
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.36 -> 1.37 --- Log message: Doxgenate comments. Add GreatestCommonDivisor64 --- Diffs of the changes: (+85 -75) MathExtras.h | 160 +++ 1 files changed, 85 inser

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-12-06 Thread Jim Laskey
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.35 -> 1.36 --- Log message: Ignoring the upper 32 bits of a 64 bit constant is not a good thing. --- Diffs of the changes: (+4 -4) MathExtras.h |8 1 files changed, 4 insertions(+), 4 deletions(-) Index:

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-08-11 Thread Chris Lattner
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.34 -> 1.35 --- Log message: remove IncludeFile turds in MathExtras.h, which bloats every .o file that #includes it. --- Diffs of the changes: (+0 -3) MathExtras.h |3 --- 1 files changed, 3 deletions(-) Index: ll

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-07-17 Thread Chris Lattner
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.32 -> 1.33 --- Log message: Add functions to compute ceil(log2(N)) to match functions for floor(log2(N)) --- Diffs of the changes: (+14 -1) MathExtras.h | 15 ++- 1 files changed, 14 insertions(+), 1 dele

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-06-21 Thread Reid Spencer
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.31 -> 1.32 --- Log message: Whoops, missed a couple more C-style casts. --- Diffs of the changes: (+2 -2) MathExtras.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Index: llvm/include/llvm/Support/Ma

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-06-21 Thread Reid Spencer
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.30 -> 1.31 --- Log message: Use C++ style casts instead of C-style casts to shut up compiler warnings when compiling with -pedantic. Passes regression tests on Linux. --- Diffs of the changes: (+18 -6) MathExtras.h |

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h SlowOperationInformer.h

2006-05-24 Thread Reid Spencer
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.29 -> 1.30 SlowOperationInformer.h updated: 1.5 -> 1.6 --- Log message: For PR786: http://llvm.cs.uiuc.edu/PR786 : Minor tweaks in public headers and a few .cpp files so that LLVM can build successfully with -pedantic and p

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-01-13 Thread Jeff Cohen
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.28 -> 1.29 --- Log message: Fix generous source of VC++ truncation warnings. --- Diffs of the changes: (+2 -2) MathExtras.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Index: llvm/include/llvm/Suppor

[llvm-commits] CVS: llvm/include/llvm/Support/MathExtras.h

2006-01-13 Thread Nate Begeman
Changes in directory llvm/include/llvm/Support: MathExtras.h updated: 1.27 -> 1.28 --- Log message: Add bswap intrinsics as documented in the Language Reference --- Diffs of the changes: (+26 -0) MathExtras.h | 26 ++ 1 files changed, 26 insertions(+) Index: llv