Implement nacl_irt_clock for non-sfi mode.

components/nacl/loader/nonsfi/abi_conversion.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_NACL_LOADER_NONSFI_ABI_CONVERSION_H_
 #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);
 
 }  // namespace nonsfi
 }  // namespace nacl
 
 #endif  // COMPONENTS_NACL_LOADER_NONSFI_ABI_CONVERSION_H_