This looks fine to me. Just one question: since we are setting -std=c++14,
is there no define for -DCXX14 instead of having to set -DCXX11? If this is
clang policy, then LGTM, otherwise, why are we not adding a define for
C++14?


http://reviews.llvm.org/D10233

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to