This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[PATCH] Fix handling of __start/__stop symbols
- From: Davide Italiano <dccitaliano at gmail dot com>
- To: binutils at sourceware dot org
- Date: Sun, 31 May 2015 13:22:11 -0700
- Subject: [PATCH] Fix handling of __start/__stop symbols
- Authentication-results: sourceware.org; auth=none
If the section contains 'Q' in the name, is_cident() returns false,
__start and __stop symbols for the section are not generated..
This is likely wrong. While at it, I changed a trailing whitespace in
the line immediately above the one containing the typo.
Sorry if I missed something in the patch submission, it's my first one
and I'm not still really familiar with the process.
Thanks,
--
Davide
% git diff
diff --git a/gold/gold.h b/gold/gold.h
index ef95f53..45292b7 100644
--- a/gold/gold.h
+++ b/gold/gold.h
@@ -255,8 +255,8 @@ inline bool
is_cident(const char* name)
{
return (name[strspn(name,
- ("0123456789"
- "ABCDEFGHIJKLMNOPWRSTUVWXYZ"
+ ("0123456789"
+ "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
"abcdefghijklmnopqrstuvwxyz"
"_"))]
== '\0');