[PATCH] D79951: [ProfileSummary] Add the PartialProfileRatio field in ProfileSummary metadata.

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 12:37:08 PDT 2020


yamauchi marked an inline comment as done.
yamauchi added inline comments.


================
Comment at: llvm/lib/IR/ProfileSummary.cpp:195
 
-  int i = 0;
-  auto &FormatMD = Tuple->getOperand(i++);
+  unsigned I = 0;
+  auto &FormatMD = Tuple->getOperand(I++);
----------------
yamauchi wrote:
> davidxl wrote:
> > can you commit the style changes as NFC first?
> Sure, will do.
Split it into D80230.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79951/new/

https://reviews.llvm.org/D79951





More information about the llvm-commits mailing list