[clang] [NFC][Clang] Use range for loops in ClangDiagnosticsEmitter (PR #115573)

[Rahul Joshi via cfe-commits]

================
@@ -1553,15 +1526,13 @@ static void emitDiagSubGroups(std::map<std::string, GroupInfo> &DiagsInGroup,
                               RecordVec &GroupsInPedantic, raw_ostream &OS) {
   OS << "static const int16_t DiagSubGroups[] = {\n"
      << "  /* Empty */ -1,\n";
-  for (auto const &I : DiagsInGroup) {
-    const bool IsPedantic = I.first == "pedantic";
-
-    const std::vector<std::string> &SubGroups = I.second.SubGroups;
+  for (auto const &[Name, Group] : DiagsInGroup) {
----------------
jurahul wrote:

Here, the & is so prevent making a copy, but I guess what you are saying is that it won't be a copy but a reference even without the &? 

https://github.com/llvm/llvm-project/pull/115573