rjmccall added inline comments. ================ Comment at: lib/Driver/Tools.cpp:1477 @@ +1476,3 @@ + // If unspecified, choose the default based on the platform. + if (ABI == ppc::FloatABI::Invalid) { + ABI = ppc::FloatABI::Hard; ---------------- hfinkel wrote: > Don't need the { } here. We don't seem to have a specific style guideline *mandating* the uses of braces around even single-line statements, but please don't *discourage* them.
http://reviews.llvm.org/D13351 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits