[PATCH] D118288: [gn build] Move -fno-builtin to crt_code

[Arthur Eubanks via Phabricator via llvm-commits]

This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGf73f367f7266: [gn build] Move -fno-builtin to crt_code (authored by aeubanks).

Changed prior to commit:
  https://reviews.llvm.org/D118288?vs=403417&id=403461#toc

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118288

Files:
  llvm/utils/gn/build/BUILD.gn
  llvm/utils/gn/secondary/compiler-rt/lib/asan/BUILD.gn
  llvm/utils/gn/secondary/compiler-rt/lib/msan/BUILD.gn
  llvm/utils/gn/secondary/compiler-rt/lib/tsan/rtl/BUILD.gn


Index: llvm/utils/gn/secondary/compiler-rt/lib/tsan/rtl/BUILD.gn
===================================================================
--- llvm/utils/gn/secondary/compiler-rt/lib/tsan/rtl/BUILD.gn
+++ llvm/utils/gn/secondary/compiler-rt/lib/tsan/rtl/BUILD.gn
@@ -128,10 +128,6 @@
     sources += [ "tsan_rtl_s390x.S" ]
   }
 
-  # FIXME: have SANITIZER_COMMON_CFLAGS thingy? should fno-builtin be in
-  # crt_code?
-  cflags += [ "-fno-builtin" ]
-
   # FIXME: link rt dl m pthread log
   # FIXME: dep on libcxx-headers?
   # FIXME: add_sanitizer_rt_version_list (cf hwasan)
Index: llvm/utils/gn/secondary/compiler-rt/lib/msan/BUILD.gn
===================================================================
--- llvm/utils/gn/secondary/compiler-rt/lib/msan/BUILD.gn
+++ llvm/utils/gn/secondary/compiler-rt/lib/msan/BUILD.gn
@@ -39,11 +39,7 @@
     "msan_thread.h",
   ]
 
-  # FIXME: have SANITIZER_COMMON_CFLAGS thingy? should fno-builtin be in
-  # crt_code?
-  cflags = [ "-fno-builtin" ]
-
-  cflags += [ "-fPIE" ]
+  cflags = [ "-fPIE" ]
 
   # Prevent clang from generating libc calls.
   cflags += [ "-ffreestanding" ]
Index: llvm/utils/gn/secondary/compiler-rt/lib/asan/BUILD.gn
===================================================================
--- llvm/utils/gn/secondary/compiler-rt/lib/asan/BUILD.gn
+++ llvm/utils/gn/secondary/compiler-rt/lib/asan/BUILD.gn
@@ -102,9 +102,7 @@
     sources += [ "asan_interceptors_vfork.S" ]
   }
 
-  # FIXME: have SANITIZER_COMMON_CFLAGS thingy? should fno-builtin be in
-  # crt_code?
-  cflags = [ "-fno-builtin" ]
+  cflags = []
   if (target_os != "win") {
     cflags += [ "-ftls-model=initial-exec" ]
   }
Index: llvm/utils/gn/build/BUILD.gn
===================================================================
--- llvm/utils/gn/build/BUILD.gn
+++ llvm/utils/gn/build/BUILD.gn
@@ -444,6 +444,7 @@
     "-funwind-tables",
     "-gline-tables-only",
     "-fvisibility=hidden",
+    "-fno-builtin",
   ]
   if (is_clang) {
     cflags += [


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118288.403461.patch
Type: text/x-patch
Size: 1986 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220127/6041fa7b/attachment.bin>