Implement nacl_irt_clock for non-sfi mode.

components/nacl/loader/nonsfi/abi_conversion.h

Index: components/nacl/loader/nonsfi/abi_conversion.h
diff --git a/components/nacl/loader/nonsfi/abi_conversion.h b/components/nacl/loader/nonsfi/abi_conversion.h
index 13479e27eb489a9cd4c8dfe437b8774a9e345d3c..771f2dcff650e8e088c5b844f15477ddef2707e3 100644
--- a/components/nacl/loader/nonsfi/abi_conversion.h
+++ b/components/nacl/loader/nonsfi/abi_conversion.h
@@ -6,11 +6,22 @@
 #define COMPONENTS_NACL_LOADER_NONSFI_ABI_CONVERSION_H_
 
 struct stat;
+struct timespec;
 struct nacl_abi_stat;
+struct nacl_abi_timespec;
 
 namespace nacl {
 namespace nonsfi {
 
+// Converts the timespec struct from NaCl's to host's ABI.
+void NaClAbiTimeSpecToTimeSpec(const struct nacl_abi_timespec& nacl_timespec,

[Mark Seaborn, 2014/01/15 19:49:28]

You might consider inlining these two functions, since they're trivial.

[hidehiko, 2014/01/16 05:30:12]

Personally, I like the current style to keep the header as simple as possible.
WDYT? Or, do you mean, removing these functions, and copy'n'pasting body of
these functions to the caller rather than sharing them?

[Mark Seaborn, 2014/01/16 19:14:13]

That's fine; it was only a suggestion.
No, I meant putting the definition in the header with an "inline" declaration.

+                               struct timespec* host_timespec);
+
+// Converts the timespec struct from host's to NaCl's ABI.
+void TimeSpecToNaClAbiTimeSpec(const struct timespec& host_timespec,
+                               struct nacl_abi_timespec* nacl_timespec);
+
+
 // Converts the stat struct from host's to NaCl's ABI.
 void StatToNaClAbiStat(
     const struct stat& host_stat, struct nacl_abi_stat* nacl_stat);