[mpich-discuss] [PATCH] config: change PAC_FUNC_NEEDS_DECL to check by assignment

Pavan Balaji balaji at mcs.anl.gov
Sat May 18 15:23:57 CDT 2013


On 05/18/2013 01:32 PM US Central Time, Jed Brown wrote:
> Pavan Balaji <balaji at mcs.anl.gov> writes:
> 
>> Last time I tried it, it didn't work in some cases.  I'll need to go
>> back and look.
> 
> Okay, then maybe there is a reason to use my version:
> 
>   void (*fptr)(void) = (void(*)(void))the_function_symbol;
> 
> Let me know what you find out because our tests had the same issue and
> I'd like to resolve it in the most robust way possible.

My testing so far seems to work fine with this patch.  I'm going to push
this in for now.  This will anyway go through a more thorough round of
testing before the next release.  If we find an issue at that point, we
can come back to this discussion.

 -- Pavan

-- 
Pavan Balaji
http://www.mcs.anl.gov/~balaji



More information about the discuss mailing list