[PATCH] D13351: [Power PC] add soft float support for ppc32

hfinkel@anl.gov via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 10 10:35:34 PST 2015


hfinkel 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;
----------------
rjmccall wrote:
> hfinkel wrote:
> > rjmccall wrote:
> > > 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.
> > I don't believe this represents the current consensus convention on this matter. We do actively discourage use of unnecessary braces for ifs, fors, etc. The exception being that a series of ifs and elses should have braces for all of them if any of them need them.
> > 
> > I often point people at the examples here as representative: http://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return
> > 
> > You're right, however, we don't formally dictate this policy in our coding standards. Do you actively dislike this viewpoint?
> The coding style document uses both styles fairly interchangeably; I would not say it states a consensus.  And yes, I actively encourage people to add braces in code reviews.
Fair enough; Strahinja, please keep the braces :-)


http://reviews.llvm.org/D13351





More information about the cfe-commits mailing list