This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch, microblaze, bfd/gas/include] Add MicroBlaze TLS Support


On 11/22/2012 04:06 AM, David Holsgrove wrote:

Add support for handling TLS symbol suffixes and generating TLS relocs for General Dynamic and Local Dynamic models.

Check in generated files that have changed as a result of new
TLS Relocs

+check_unique_offset(bfd_vma offset, int tls_type, bfd_vma v) +{ + static bfd_vma offsets[1000]; + static bfd_vma values[1000];

Why 1000?

+  for (scan1 = 0; scan1 < noffsets; scan1++)
+  {
+	if (offsets[scan1] == offset)
+	  {
+		fprintf (stderr, "Duplicate Offset: %lx type: %x Old: %lx New: %lx \n",
+                     offset, tls_type, values[scan1], v);
+	  }
+  }
+  fprintf (stderr, "%d Registered Offset: %lx Value: %lx type: %x\n",
+                   noffsets, offset, v, tls_type);

Please fix indent. There are several places where the indent is incorrect.

Is this debugging code?  Are the messages errors which you expect to
generate?  If yes, use bfd_perror().

+ if ( IS_TLS_LD (tls_type) )

No spaces after/before parens surrounding expression.  There are other places
which have extra spaces.

+ abort();

Space before paren in function calll.  There are other places which need spaces
before parens.

+ check_unique_offset( off, tls_type, 0xDDDDDDDD);

What is 0xDDDDDDDD?

+      size *= (sizeof (*local_got_refcounts)
+               + sizeof (*local_got_tls_masks));

One line.


-- Michael Eager eager@eagercon.com 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]