[PATCH] D44519: Add llvm-exegesis tool.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: tools/llvm-exegesis/lib/InstructionSnippetGenerator.cpp:159
+  // Adding variables to the graph.
+  for (size_t I = 0; I < Vars.size(); ++I) {
+    const Variable &Var = Vars[I];
----------------
courbet wrote:
> RKSimon wrote:
> > ```
> > for (size_t I = 0, E = Vars.size(); I < E; ++I) {
> > ```
> The compiler should be able to inline these, right ?
https://llvm.org/docs/CodingStandards.html#don-t-evaluate-end-every-time-through-a-loop


Repository:
  rL LLVM

https://reviews.llvm.org/D44519