Looks reasonable (though your description of two separate commits was a bit confusing, since the code review doesn't break it out/make it clear which parts are which) http://reviews.llvm.org/D10367 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/