[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:

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

[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 +--

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 @@ ///

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 directory

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 argument, so there's no

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 its

[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-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-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

[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-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-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

[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:

[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

[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

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-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

[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:

[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: