rsmith added inline comments.

================
Comment at: lib/Basic/SourceLocation.cpp:89-93
+  B.print(OS, SM);
+  OS << ",\n ";
+  E.print(OS, SM);
+  OS << "]\n";
+}
----------------
It would seem somewhat more natural to dump `SourceRange`s the same way we dump 
them in `-ast-dump`. (That is: use angle brackets, no newline, and omit the 
file name / line number / column number on the end location if it's the same as 
on the start location.)


Repository:
  rC Clang

https://reviews.llvm.org/D50662



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

Reply via email to