No tests change, and i'm not sure how to test this, but it's better safe than sorry.
I'm not sure what this FIXME was trying to say. getNode should detect the constants and call FoldConstantArithmetic anyway. @spatel?
LGTM. I dont' think this causes a functional change. At best its a very minor compile time improvement.