Hi Vladimir, This looks OK to me. The string switching is looking like a StringSwitch might be almost useful, but I can't see enough benefit to insist on it.
Cheers, James On Wed, 1 Apr 2015 at 14:17 Vladimir Sukharev <[email protected]> wrote: > REPOSITORY > rL LLVM > > http://reviews.llvm.org/D8493 > > EMAIL PREFERENCES > http://reviews.llvm.org/settings/panel/emailpreferences/ > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
