[llvm-commits] [llvm] r48676 - in /llvm/trunk: include/llvm/System/Memory.h lib/System/Unix/Memory.inc lib/System/Win32/Memory.inc

[Duncan Sands]

Hi Owen,

>  
> +      static MemoryBlock AllocateRW(unsigned NumBytes,
> +                                    const MemoryBlock *NearBlock,
> +                                    std::string *ErrMsg = 0);
> +
>        /// This method releases a block of Read/Write/Execute memory that was
>        /// allocated with the AllocateRWX method. It should not be used to
>        /// release any memory block allocated any other way.

how about having the comment explaining what it does in the header?  That means
you don't have to rootle around finding where the routine is implemented just
to find out what it is supposed to do.

Ciao,

Duncan.