I was away the last few days, so just found the changes now.

IMO, it's a good idea to use a new module for the new compiler, but don't think it's a good idea to make the whole module private, since this implicitly disallows sub-classing the compiler class going forward, which people will need to do sooner or later.

Why not rename the module to msvc14compiler (or some other public name) instead ?