[PATCH] D23387: [Analyzer] Report found fields order in PaddingChecker.

[Ben Craig via cfe-commits]

bcraig added a comment.

LGTM.  Thanks for the patch!


================
Comment at: lib/StaticAnalyzer/Checkers/PaddingChecker.cpp:217
@@ +216,3 @@
+        // then large field indices to small field indices
+        return std::make_tuple(Align, -Size,
+                               Field ? -static_cast<int>(Field->getFieldIndex())
----------------
Nit:
I think std::tie is the more idiomatic way to do this (not that I had it right before either).  You get less copying of values, though that doesn't matter much for the types we are using here.


https://reviews.llvm.org/D23387