CodeSourcery > List Archives

Actions

icon Post
text/html Subscribe
text/html Unsubscribe
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [vsipl++] Compiling VSIPL++ on Cell PPE

  • To: Jules Bergmann <jules@xxxxxxxxxxxxxxxx>
  • Subject: Re: [vsipl++] Compiling VSIPL++ on Cell PPE
  • From: Hahn Kim <hgk@xxxxxxxxxx>
  • Date: Tue, 9 Oct 2007 13:57:06 -0400
<HTML><BODY style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; ">Hi Jules,<DIV><BR class="khtml-block-placeholder"></DIV><DIV>We took a look at the PS3 sal.h over here and compared it to our Mercury CTES sal.h and it appears you are correct; SAL's API has been updated to explicitly require "signed char *" instead of "char *".  According to the comments in the file, this was changed on June 15, 2007.  It also appears that this change pops up in a bunch of places.</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>I'm sending along the entire sal.h for your reference.  I'm guessing the build of SAL we have for our PS3 is an advance copy.</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>If we could get a snapshot, that would be great.  We'll also notify Mercury that we encountered this problem, as you suggested.  Thanks!</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV>Hahn</DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV></DIV></BODY></HTML>

Attachment: sal.h
Description: Binary data

<HTML><BODY style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; "><DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV><BR class="khtml-block-placeholder"></DIV><DIV><BR><DIV><DIV>On Oct 9, 2007, at 12:30 PM, Jules Bergmann wrote:</DIV><BR class="Apple-interchange-newline"><BLOCKQUOTE type="cite"><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Hahn,</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">As far as I can tell, it looks like Mercury changed their API. According to the latest SAL documentation,</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><A href="https://customers.mc.com/self_solve/docs/doc_files/SAL_Reference.pdf";>https://customers.mc.com/self_solve/docs/doc_files/SAL_Reference.pdf</A></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">vconvert_s8_f32x takes a 'char*' first argument:</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">void vconvert_s8_f32x(</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">char *A, /* input vector */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">int i, /* address stride for A */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">float *C, /* output vector */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">int k, /* address stride for C */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">float *scale, /* pointer to scale value (NULL=1.0) */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">float *bias /* pointer to bias value (NULL=0.0) */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">int n, /* element count */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">int rfflag, /* reserved */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">int eflag ); /* ESAL flag */</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">This is consistent with our experience using SAL.</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">However, from your error message,</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV> <BLOCKQUOTE type="cite"><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">src/vsip/opt/sal/elementwise.hpp:256: error: invalid conversion from ‘char* const’ to ‘signed char*’</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">src/vsip/opt/sal/elementwise.hpp:256: error: <SPAN class="Apple-converted-space">  </SPAN>initializing argument 1 of ‘void vconvert_s8_f32x(signed char*, int, float*, int, float*, float*, int, int, int)’</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">make: *** [src/vsip/initfin.o] Error 1</DIV> </BLOCKQUOTE><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">it now appears SAL expects a 'signed char*'.<SPAN class="Apple-converted-space">  </SPAN>You might look at sal.h to confirm this change.<SPAN class="Apple-converted-space">  </SPAN>If so, you might also let Mercury know, since they probably didn't mean to break application compatibility like this.</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">We can of course work around this in VSIPL++.<SPAN class="Apple-converted-space">  </SPAN>The easiest way for us to fix this by sending you an updated snapshot.<SPAN class="Apple-converted-space">  </SPAN>However, if it is more convenient for you, we can also send you a patch against 1.3 or 1.3.1. Let me know which you would prefer.</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><SPAN class="Apple-tab-span" style="white-space:pre">	</SPAN><SPAN class="Apple-tab-span" style="white-space:pre">	</SPAN><SPAN class="Apple-tab-span" style="white-space:pre">	</SPAN><SPAN class="Apple-tab-span" style="white-space:pre">	</SPAN>-- Jules</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">--<SPAN class="Apple-converted-space"> </SPAN></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Jules Bergmann</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">CodeSourcery</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><A href="mailto:jules@xxxxxxxxxxxxxxxx";>jules@xxxxxxxxxxxxxxxx</A></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">(650) 331-3385 x705</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: 14px; "><BR></DIV> </BLOCKQUOTE></DIV><BR><DIV> <SPAN class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; color: rgb(0, 0, 0); font-family: Courier; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><SPAN class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; color: rgb(0, 0, 0); font-family: Courier; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">-- </DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Hahn Kim</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">MIT Lincoln Laboratory   Phone: (781) 981-0940</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">244 Wood Street, S2-252  Fax: (781) 981-5255</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Lexington, MA 02420      E-mail: <A href="mailto:hgk@xxxxxxxxxx";>hgk@xxxxxxxxxx</A></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><BR class="khtml-block-placeholder"></DIV><BR class="Apple-interchange-newline"></SPAN></SPAN> </DIV><BR></DIV></DIV></BODY></HTML>
© 2004-2009 CodeSourcery. All Rights Reserved. Legal Information (888) 776-0262 · (650) 331-3385