I have a comment only on your last choice: i don't know the feature history of valgrind but i believe that there were versions which had the file memcheck.h without the mentioned macro.

I would like to leave the code that handles this issue like it was in the original patch (if it is fine with you).

I checked a couple of older valgrind releases, and you are correct. There are versions where it is undefined. I've reverted this change back to match your original patch. Thanks.

- Sean
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to