[PATCH] D34027: [XRay] Fix computation of function size subject to XRay threshold

[Dean Michael Berris via Phabricator via llvm-commits]

dberris accepted this revision.
dberris added a comment.
This revision is now accepted and ready to land.

Thanks @rSerge -- just one suggestion for readability.



================
Comment at: lib/CodeGen/XRayInstrumentation.cpp:146-148
+    typedef MachineFunction::const_iterator BasicBlockIterator;
+    for (BasicBlockIterator it = MF.begin(); it != MF.end(); it++)
+      MICount += it->size();
----------------
Could this be cleaner as a range-based for-loop?

```
int64_t MICount = 0;
for (const auto& MBB : MF)
  MICount += MBB.size();
```


https://reviews.llvm.org/D34027