Rename runOnBasicBlock to eliminateDeadStores. Add an overload that takes a Function instead of BasicBlock. Made all the helpers static. Cleaned up function description comments. clang-format.

Pull out all the member functions to be helpers. The diff here is less clean but this is just because the helpers needed to be moved around.

clang-format.

The meaning/name of the flags have been changed so that we only pass them to disable the warnings. The flags also no longer interfere with the statistics.

Great, if possible Sean or David could you commit this for me? I don't have
commit access.

Not sure if the amount of change here is 'ugly' but I've addressed all the responses.

This is great! This actually fixes a problem case I ran into some months ago in the wild, where a memset wasn't being wiped away despite being fully overwritten by the stores following it. Example shown below:

If you could, please.

I think it was able to avoid test failure simply because there are so few elements in the existing test that compareNames in qsort just happened to produce the same results each time.

When iterating with a set of passes manually, I found that GlobalOpt was not reaching a fixed point but was not reporting any changes.