From f500895efe84525c131beadd2e83b30caa6674a3 Mon Sep 17 00:00:00 2001 From: alaviss Date: Thu, 23 Jan 2020 12:45:31 +0000 Subject: Unexport even more symbols (#13214) * system/gc: don't export markStackAndRegisters * compiler/cgen: unexport internal symbols As these functions are Nim-specific walkaround against C's optimization schemes, they don't serve any purpose being exported. * compiler/cgen: don't export global var unless marked * compiler/ccgthreadvars: don't export threadvar unless marked * tests/dll/visibility: also check for exports This ensure that these changes don't break manual exports. * compiler/cgen: hide all variables created for constants * compiler/ccgtypes: don't export RTTI variables * compiler/ccgexprs: make all complex const static * nimbase.h: fix export for windows * compiler/cgen, ccgthreadvars: export variables correctly For C/C++ variables, `extern` means that the variable is defined in an another unit. Added a new N_LIB_EXPORT_VAR to correctly export variables. --- tests/dll/visibility.nim | 36 ++++++++++++++++++++++++++++++------ 1 file changed, 30 insertions(+), 6 deletions(-) (limited to 'tests/dll/visibility.nim') diff --git a/tests/dll/visibility.nim b/tests/dll/visibility.nim index 7341e3311..ec15dad29 100644 --- a/tests/dll/visibility.nim +++ b/tests/dll/visibility.nim @@ -1,6 +1,5 @@ discard """ - output: "could not import: foo" - exitcode: 1 + output: "" """ const LibName {.used.} = @@ -12,8 +11,33 @@ const LibName {.used.} = "libvisibility.so" when compileOption("app", "lib"): - proc foo() {.exportc.} = - echo "failed" + var + bar {.exportc.}: int + thr {.exportc, threadvar.}: int + proc foo() {.exportc.} = discard + + var + exported {.exportc, dynlib.}: int + exported_thr {.exportc, threadvar, dynlib.}: int + proc exported_func() {.exportc, dynlib.} = discard elif isMainModule: - proc foo() {.importc, dynlib: LibName.} - foo() + import dynlib + + let handle = loadLib(LibName) + + template check(sym: untyped) = + const s = astToStr(sym) + if handle.symAddr(s) != nil: + echo s, " is exported" + template checkE(sym: untyped) = + const s = astToStr(sym) + if handle.symAddr(s) == nil: + echo s, " is not exported" + + check foo + check bar + check thr + + checkE exported + checkE exported_thr + checkE exported_func -- cgit 1.4.1-2-gfad0 : root/tests/stdlib/tsugar.nim
blob: 2ea96cfbbf6e740d63268509ba2792fac1994815 (plain) (tree) 
f4449a897 ^

078495c79 ^

4fa86422c ^

f4449a897 ^




2d9af2bd5 ^

3d2f0e2c7 ^

f4449a897 ^

cc2b0f017 ^




cecaf9c56 ^

















f4449a897 ^










































































cc2b0f017 ^
























































f4449a897 ^












bc1db0d6f ^

f4449a897 ^























































bc1db0d6f ^

f4449a897 ^












































f4449a897 ^



078495c79 ^


f4449a897 ^




078495c79 ^

f4449a897 ^







2d9af2bd5 ^






f46f26e79 ^


2d9af2bd5 ^




cecaf9c56 ^


078495c79 ^




f4449a897 ^

1

2

3

4
5
6
7

8

9

10

11
12
13
14

15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31

32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105

106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161

162
163
164
165
166
167
168
169
170
171
172
173

174

175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229

230

231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274

275
276
277

278
279

280
281
282
283

284

285
286
287
288
289
290
291

292
293
294
295
296
297

298
299

300
301
302
303

304
305

306
307
308
309

310