Android doesn't have posix_memalign(), so use memalign()

Android doesn't have posix_memalign(), so use memalign()
there instead.

BUG=http://code.google.com/p/nativeclient/issues/detail?id=3032
TEST=Compiles, works
Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=10198

[Mark Seaborn, 2012-11-02 15:35:57]

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
File src/shared/platform/posix/aligned_malloc.c (left):

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
src/shared/platform/posix/aligned_malloc.c:11: 
Don't change the spacing unnecessarily, please.  Two lines after the #includes
was intentional.

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
File src/shared/platform/posix/aligned_malloc.c (right):

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
src/shared/platform/posix/aligned_malloc.c:13: return memalign(alignment, size);
Android added posix_memalign() in December 2011 in
https://android.googlesource.com/platform/bionic.git/+/85aad909560508410101c1....

What Android versions are you targeting?  Please add a comment here saying which
old version you're targeting that didn't have posix_memalign() and add a TODO to
remove this when the old version is obsolete.

Also comment that, in general, memalign() is not guaranteed to return a pointer
that can be passed to free(), but it does with Android's Bionic C library.

[Nikolay, 2012-11-06 11:09:05]

PTAL.

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
File src/shared/platform/posix/aligned_malloc.c (left):

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
src/shared/platform/posix/aligned_malloc.c:11: 
On 2012/11/02 15:35:57, Mark Seaborn wrote:
> Don't change the spacing unnecessarily, please.  Two lines after the #includes
> was intentional.

Done.

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
File src/shared/platform/posix/aligned_malloc.c (right):

https://codereview.chromium.org/11364047/diff/1/src/shared/platform/posix/ali...
src/shared/platform/posix/aligned_malloc.c:13: return memalign(alignment, size);
On 2012/11/02 15:35:57, Mark Seaborn wrote:
> Android added posix_memalign() in December 2011 in
>
https://android.googlesource.com/platform/bionic.git/+/85aad909560508410101c1....
> 
> What Android versions are you targeting?  Please add a comment here saying
which
> old version you're targeting that didn't have posix_memalign() and add a TODO
to
> remove this when the old version is obsolete.
> 
> Also comment that, in general, memalign() is not guaranteed to return a
pointer
> that can be passed to free(), but it does with Android's Bionic C library.

Done.

[Mark Seaborn, 2012-11-06 13:54:44]

LGTM

https://codereview.chromium.org/11364047/diff/1004/src/shared/platform/posix/...
File src/shared/platform/posix/aligned_malloc.c (right):

https://codereview.chromium.org/11364047/diff/1004/src/shared/platform/posix/...
src/shared/platform/posix/aligned_malloc.c:16: * luckily returns free()'able
memory.
How about: "luckily, in Bionic, returns free()'able memory, although it is not
required to in general."