erichkeane added inline comments.

================
Comment at: lib/CodeGen/TargetInfo.cpp:1688
+    for (auto &I : FI.arguments()) {
+      if(Count < 6)
+        I.info = reclassifyHvaArgType(I.type, State, I.info);
----------------
majnemer wrote:
> erichkeane wrote:
> > majnemer wrote:
> > > Formatting.
> > I don't see what you mean?  This area went through clang-format?  What did 
> > I miss?
> Your code has "if(Count < 6)", I'm incredibly surprised that clang-format 
> would format your code like this. There should be a space after the if and 
> before the left-parenthesis.
Ah, thanks!  I do go through my patches and 'ctrl-K' for clang-format, so I 
don't know if I missed this section or what, but I've fixed it as a part of the 
other fixes here, and the re-clang-formatting took care of it.  Thanks for the 
clarification!


https://reviews.llvm.org/D27529



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to