* fix extra alignment for Xtensa literal_position directive
@ 2006-08-28 0:32 Bob Wilson
0 siblings, 0 replies; only message in thread
From: Bob Wilson @ 2006-08-28 0:32 UTC (permalink / raw)
To: binutils
[-- Attachment #1: Type: text/plain, Size: 643 bytes --]
I've committed this patch to fix a minor problem when using the Xtensa
absolute_literals directive. The literal_position directive was inadvertently
inserting an alignment to a word boundary when the absolute_literals mode was
enabled. Aside from an erroneous conditional, it turns out that this alignment
is redundant in all cases, so the patch removes it.
2006-08-25 Sterling Augustine <sterling@tensilica.com>
Bob Wilson <bob.wilson@acm.org>
* config/tc-xtensa.c (xtensa_mark_literal_pool_location): Do not check
the state of the absolute_literals directive. Remove align frag at
the start of the literal pool position.
[-- Attachment #2: gas-lit-position.patch --]
[-- Type: text/plain, Size: 736 bytes --]
Index: config/tc-xtensa.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-xtensa.c,v
retrieving revision 1.69
diff -u -p -r1.69 tc-xtensa.c
--- config/tc-xtensa.c 25 Aug 2006 00:08:55 -0000 1.69
+++ config/tc-xtensa.c 25 Aug 2006 19:56:07 -0000
@@ -4516,12 +4516,9 @@ xtensa_mark_literal_pool_location (void)
emit_state s;
fragS *pool_location;
- if (use_literal_section && !directive_state[directive_absolute_literals])
+ if (use_literal_section)
return;
- frag_align (2, 0, 0);
- record_alignment (now_seg, 2);
-
/* We stash info in these frags so we can later move the literal's
fixes into this frchain's fix list. */
pool_location = frag_now;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2006-08-25 20:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-08-28 0:32 fix extra alignment for Xtensa literal_position directive Bob Wilson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).