Build nacl_helper_nonsfi with GN

build/config/nacl/BUILD.gn

 # Copyright (c) 2014 The Native Client Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/nacl/config.gni")
 
 # Native Client Definitions
 config("nacl_defines") {
   if (is_linux || is_android || is_nacl) {
     defines = [
@@ skipping 20 matching lines @@
 source_set("nacl_base") {
   public_configs = [ ":nacl_defines" ]
   if (current_os == "nacl") {
     public_configs += [ ":nexe_defines" ]
   }
 }
 
 config("compiler") {
   configs = []
   cflags = []
+  ldflags = []
 
   if (is_clang && current_cpu != "pnacl") {
     # -no-integrated-as is the default in nacl-clang for historical
     # compatibility with inline assembly code and so forth.  But there
     # are no such cases in Chromium code, and -integrated-as is nicer in
     # general.  Moreover, the IRT must be built using LLVM's assembler
     # on x86-64 to preserve sandbox base address hiding.  Use it
     # everywhere for consistency (and possibly quicker builds).
     cflags += [ "-integrated-as" ]
   }
+  if (is_nacl_nonsfi) {
+    cflags += [
+      "--pnacl-allow-native",

[Mark Seaborn, 2015/10/22 21:48:30]

Is there a reason that some of these flags are in this file while others are in
build/config/compiler/BUILD.gn?

I suppose there is an intentional split from config("compiler_cpu_abi")?

[Petr Hosek, 2015/10/23 03:01:15]

Yes, I tried to follow the existing conventions.

+      "--pnacl-allow-translate",
+    ]
+    ldflags += [ "--pnacl-allow-native" ]
+    include_dirs = [ "//native_client/src/public/linux_syscalls" ]
+  }
 
   asmflags = cflags
 }
 
 config("compiler_codegen") {
   cflags = []
 
   if (is_nacl_irt) {
     cflags += [
       # A debugger should be able to unwind IRT call frames.  This is
@@ skipping 25 matching lines @@
     "-Os",
 
     # These are omitted from non-IRT libraries to keep the libraries
     # themselves small.
     "-ffunction-sections",
     "-fdata-sections",
   ]
 
   ldflags = [ "-Wl,--gc-sections" ]
 }