clang doesn't propagate attributes like "asm labels" and "visibility(hidden)"

to builtins like mem{set,cpy,move} and __stack_smash_handler.  So, when
building with clang, instead mark those as protected visibility to get rid
of the PLT relocations.  We can't take the address of them then, but that's
ok: it's a build-time error not a run-time error.

ok kettenis@

1 files changed, 2 insertions, 2 deletions

diff --git a/src/lib/libc/string/memset.c b/src/lib/libc/string/memset.c
index 242529ee0f..0c261f0965 100644
--- a/src/lib/libc/string/memset.c
+++ b/src/lib/libc/string/memset.c
@@ -1,4 +1,4 @@
-/*      $OpenBSD: memset.c,v 1.7 2015/08/31 02:53:57 guenther Exp $ */
+/*      $OpenBSD: memset.c,v 1.8 2017/11/29 05:13:57 guenther Exp $ */
 /*-
  * Copyright (c) 1990 The Regents of the University of California.
  * All rights reserved.
@@ -45,4 +45,4 @@ memset(void *dst, int c, size_t n)
         }
         return (dst);
 }
-DEF_STRONG(memset);
+DEF_BUILTIN(memset);