[PATCH] D104019: [flang] Add initial implementation for CPU_TIME

Jean Perier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 11 01:28:36 PDT 2021


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

LGTM, thanks !



================
Comment at: flang/unittests/RuntimeGTest/Time.cpp:19
+  // Doing some writes to a volatile variable should do the trick.
+  for (int i = 0; i < (1 << 8); ++i)
+    x = i;
----------------
nit: other runtime test files also uses brace around single line for. Only lib/[Optmizer|Lower],  and include/flang/[Optmizer|Lower] files are using llvm convention there.


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

https://reviews.llvm.org/D104019



More information about the llvm-commits mailing list