Hi Hal,

I saw that you didn't add this patch yet.
Using the -fno-strict-aliasing falg avoids the compiler from doing some 
optimizations that might cause the strict aliasing
bugs. So this is a fix, and not a workaround to remove the warnings.
I agree that it is not the best fix. It is better to fix the code in such a way 
that will enable these optimizations too.
I can look into it and have a better fix, but it will take some time.
Until then, this fix should be added, since currently we can produce code that 
might have these problems in it.
Here is the patch again (in case you lost it along the mails....)

Thanks,
Yael




Signed-off-by:  Yael Kalka <[EMAIL PROTECTED]>

Index: opensm/Makefile.am
===================================================================
--- opensm/Makefile.am  (revision 3487)
+++ opensm/Makefile.am  (working copy)
@@ -64,7 +64,7 @@ opensm_SOURCES = main.c osm_db_files.c o
                 osm_ucast_mgr.c osm_ucast_updn.c \
                 osm_vl15intf.c osm_vl_arb_rcv.c\
                 osm_vl_arb_rcv_ctrl.c st.c
-opensm_CFLAGS = -Wall $(OSMV_CFLAGS) -DVENDOR_RMPP_SUPPORT $(DBGFLAGS) 
-D_XOPEN_SOURCE=600 -D_BSD_SOURCE=1
+opensm_CFLAGS = -Wall $(OSMV_CFLAGS) -fno-strict-aliasing 
-DVENDOR_RMPP_SUPPORT $(DBGFLAGS) -D_XOPEN_SOURCE=600 -D_BSD_SOURCE=1
 opensm_CXXFLAGS = -Wall $(OSMV_CFLAGS) -DVENDOR_RMPP_SUPPORT $(DBGFLAGS) 
-D_XOPEN_SOURCE=600 -D_BSD_SOURCE=1
 
 # for linking with the simulator client library we have to use g++:

_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

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

Reply via email to