------- Comment #6 from manu at gcc dot gnu dot org 2008-01-08 17:26 ------- (In reply to comment #3) > > I also counted about 300 similar cases where Intel > produced warning #593. I think there is quite a bit > of pointless computation going on. >
Thanks. I think optimisers should remove all of them so they should not have any effect. Nonetheless, if they are not really used, they should be removed from the code. > I attach a patch again 4.3 snapshot 20071228. Unfortunately, GCC 4.3 is open only for regression and documentation fixes. Your patch would need to wait for GCC 4.4. Yet, you can submit it right now and there are chances that someone will review. To avoid conflicts between maintainer of different parts of the compiler, I would suggest one different submission per component (or if you cannot distinguish the components, one submission per file). > Hint: Perhaps if this patch gets into 4.3 it would encourage > me to look at the other 300 cases ? Patches must be bootstrapped+regression tested and sent to gcc-patches. Check http://gcc.gnu.org/contribute.html for further instructions. If you need help with a particular step, write me an email. Examples of submissions can be found in the archives of gcc-patches. -- manu at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |manu at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34592
