Macros are not needed outside translation units

Macros are as local as types; we use macros for always-inlined functions, and
contained symbols never end up in the symbol table. Thus macros must be removed
before attempting to recurse into them for the purpose of symbol-table cleanup.

Author: tautschnig

regression/ansi-c/always_inline1/main.c

@@ -22,6 +22,12 @@ static inline __attribute__((always_inline)) void unused(unsigned long offset)
   // discard it
   ((void)sizeof(char[1 - 2 * !!(!__builtin_constant_p(offset))]));
 }
+
+// unused, but no 'static'
+inline __attribute__((__always_inline__)) int also_unused(int _c)
+{
+  return _c;
+}
 #endif
 
 int main()

src/linking/remove_internal_symbols.cpp

@@ -125,7 +125,7 @@ void remove_internal_symbols(
         is_file_local=false;
     }
 
-    if(is_type)
+    if(is_type || symbol.is_macro)
     {
       // never EXPORTED by itself
     }