[llvm-commits] [llvm] r162992 - in /llvm/trunk: include/llvm/Analysis/ProfileDataLoader.h lib/Analysis/ProfileDataLoader.cpp

Alastair Murray alastairmurray42 at gmail.com
Sat Sep 1 04:58:18 PDT 2012


Hi Benjamin,

Thank you for the clean-up.  I've had a read of what has changed for 
future reference.

Regards,
Alastair,

On 31/08/12 08:43, Benjamin Kramer wrote:
> Author: d0k
> Date: Fri Aug 31 07:43:07 2012
> New Revision: 162992
>
> URL: http://llvm.org/viewvc/llvm-project?rev=162992&view=rev
> Log:
> Clean up ProfileDataLoader a bit.
>
> - Overloading operator<< for raw_ostream and pointers is dangerous, it alters
>    the behavior of code that includes the header.
> - Remove unused ID.
> - Use LLVM's byte swapping helpers instead of a hand-coded.
> - Make ReadProfilingData work directly on a pointer.
>
> No functionality change.
>
> Modified:
>      llvm/trunk/include/llvm/Analysis/ProfileDataLoader.h
>      llvm/trunk/lib/Analysis/ProfileDataLoader.cpp




More information about the llvm-commits mailing list