Re: [Open64-devel] IF NODE WHIRL question

2010-10-08 Thread Yiran Wang
Agree, when the value is unwanted explicitly, the generate WHIRL is just the same, i.e. #include #include int main() { int i = rand(); printf("i = %i\n", i); if (i) printf("i = %i\n", i); return 0; } On Thu, Oct 7, 2010 at 1:02 PM, Handong Ye wrote: > A wild guess. The code needs

Re: [Open64-devel] Review request for implement three gcc builtin functions

2010-10-08 Thread Chen, Rui (Roger, TSG-GDCC-SH)
I modified the patch a little bit, defined a PU_HAS_APPLY_ARGS flag in symtab_defs.h, and set this flag in wgen. So I don’t need to traverse the whole WHIRL tree later. Pls help to review the changes. Regards, Roger From: Chen, Rui (Roger, TSG-GDCC-SH) Sent: 2010年9月25日 14:23 To: open64-devel@l

Re: [Open64-devel] Patch for bug 621, using non-standard strdupa function

2010-10-08 Thread Sun Chan
I am fine with either fixes Sun 2010/10/9 "C. Bergström" : > Steve Ellcey wrote: >> Index: osprey/driver/file_names.c >> === >> --- osprey/driver/file_names.c        (revision 3367) >> +++ osprey/driver/file_names.c        (working co

Re: [Open64-devel] Patch for bug 621, using non-standard strdupa function

2010-10-08 Thread C. Bergström
Steve Ellcey wrote: > Index: osprey/driver/file_names.c > === > --- osprey/driver/file_names.c(revision 3367) > +++ osprey/driver/file_names.c(working copy) > @@ -156,7 +156,7 @@ get_object_file (char *src) >

[Open64-devel] Patch for bug 621, using non-standard strdupa function

2010-10-08 Thread Steve Ellcey
This patch is for PR 621, which says (correctly) that strdupa is not a portable function. It is a linux extension. This patch replaces 'strdupa(src)' with 'strcpy(alloca(strlen(src)+1), src)', which accomplishes the same thing that strdupa does. I searched the sources and this is the only use of