PNaCl SIMD documentation: improve documentation on casts and __builtin_convertvector.

PNaCl SIMD documentation: improve documentation on casts and __builtin_convertvector.

R= nfullagar@chromium.org
BUG= https://code.google.com/p/nativeclient/issues/detail?id=2205
TEST= none
NOTRY=true
(documentation only change)

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266902

[nfullagar1, 2014-04-28 17:35:23]

https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
File native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
(right):

https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:300:
v4u c = __builtin_convertvector(b, v4u); /* c = {0,2,3,0} */
It would be useful if both the example and comment above include mention of
truncation in the case of float->int.  vector "b" should have at least one
element whose fractional value is > 0.5.

[JF, 2014-04-28 17:42:54]

https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
File native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
(right):

https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:300:
v4u c = __builtin_convertvector(b, v4u); /* c = {0,2,3,0} */
On 2014/04/28 17:35:23, nfullagar1 wrote:
> It would be useful if both the example and comment above include mention of
> truncation in the case of float->int.  vector "b" should have at least one
> element whose fractional value is > 0.5.

Done.

[nfullagar1, 2014-04-28 17:48:30]

On 2014/04/28 17:42:54, JF wrote:
>
https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
> File
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
> (right):
> 
>
https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
>
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:300:
> v4u c = __builtin_convertvector(b, v4u); /* c = {0,2,3,0} */
> On 2014/04/28 17:35:23, nfullagar1 wrote:
> > It would be useful if both the example and comment above include mention of
> > truncation in the case of float->int.  vector "b" should have at least one
> > element whose fractional value is > 0.5.
> 
> Done.

example yes, but also the description on builtin_convertvector above should
mention truncation

[JF, 2014-04-28 17:52:47]

On 2014/04/28 17:48:30, nfullagar1 wrote:
> On 2014/04/28 17:42:54, JF wrote:
> >
>
https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
> > File
> native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
> > (right):
> > 
> >
>
https://codereview.chromium.org/255813009/diff/1/native_client_sdk/doc_genera...
> >
>
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:300:
> > v4u c = __builtin_convertvector(b, v4u); /* c = {0,2,3,0} */
> > On 2014/04/28 17:35:23, nfullagar1 wrote:
> > > It would be useful if both the example and comment above include mention
of
> > > truncation in the case of float->int.  vector "b" should have at least one
> > > element whose fractional value is > 0.5.
> > 
> > Done.
> 
> example yes, but also the description on builtin_convertvector above should
> mention truncation

Done.

[nfullagar1, 2014-04-29 16:50:50]

one last detail to add, otherwise lgtm

https://codereview.chromium.org/255813009/diff/40001/native_client_sdk/doc_ge...
File native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
(right):

https://codereview.chromium.org/255813009/diff/40001/native_client_sdk/doc_ge...
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:294:
to convert from one type to another, truncating when converting from
...__builtin_convertvector can be used to convert from one type to another
provided both types have the same number of elements and truncating when...

[JF, 2014-04-29 16:55:38]

https://codereview.chromium.org/255813009/diff/40001/native_client_sdk/doc_ge...
File native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html
(right):

https://codereview.chromium.org/255813009/diff/40001/native_client_sdk/doc_ge...
native_client_sdk/doc_generated/reference/pnacl-c-cpp-language-support.html:294:
to convert from one type to another, truncating when converting from
On 2014/04/29 16:50:50, nfullagar1 wrote:
> ...__builtin_convertvector can be used to convert from one type to another
> provided both types have the same number of elements and truncating when...

Done.

[JF, 2014-04-29 16:55:45]

The CQ bit was checked by jfb@chromium.org

[commit-bot: I haz the power, 2014-04-29 16:56:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jfb@chromium.org/255813009/60001

[commit-bot: I haz the power, 2014-04-29 16:57:33]

Change committed as 266902