aaron.ballman added inline comments.
================
Comment at: lib/Serialization/ASTReader.cpp:5074
@@ -5087,3 +5073,3 @@
bool operator()(ModuleFile &M) {
- HeaderFileInfoLookupTable *Table
+ auto *Table
= static_cast<HeaderFileInfoLookupTable *>(M.HeaderFileInfoTable);
----------------
Eugene.Zelenko wrote:
> I don't think it's good idea to make coding practices depending on
> formatting.
> I don't think it's good idea to make coding practices depending on formatting.
I disagree. The use of auto is only when it adds to readability of the code.
We've distilled this down to cases where the type is obvious due to already
being stated in the initialization of the variable. If that initialization
means I have to look somewhere else to see the actual type, then the benefit of
auto is reduced.
http://reviews.llvm.org/D15251
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
