Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: [OMPI devel] RFC: Dead code removal
From: Jeff Squyres (jsquyres) (jsquyres_at_[hidden])
Date: 2013-07-05 09:14:06


What: Remove code identified by compilers as being dead (e.g., set-but-unused)

Why: It's dead, Jim.

Where: Attached patch removes snipits of code from:

- openib udcm CPC
- coll inter
- coll ml
- coll sm
- sbgp basemsocket
- routed binomial

Timeout: next Tuesday teleconf (July 9)

More detail:

In the spirit of George's patch, I found a few more warnings that were simple to eliminate. Most of these should be non-controversial, hence the short timeout.

If you own one of the above components, you might want to make sure that this dead code removal doesn't actually indicate a bug.

Note that I specifically did *not* remove some dead code compiler warnings from:

- oob
- osc

because these two sections are both under revamp right now, and a) it's not worth fixing them because they're changing so much, and b) it would probably create unnecessary conflicts for the maintainers of those revamps.

-- 
Jeff Squyres
jsquyres_at_[hidden]
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/