diff options
author | Alan Modra <amodra@gmail.com> | 2017-11-07 16:32:07 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2017-11-07 17:01:17 +1030 |
commit | e54e9ac577591f53bd9552210b1062c6230030be (patch) | |
tree | 8f8d216c3e5264e4caead19b1ad073f965f6c0ed /gas/read.c | |
parent | d3a49aa80b8a6eab1c684deddad3542c4cdaa497 (diff) | |
download | gdb-e54e9ac577591f53bd9552210b1062c6230030be.zip gdb-e54e9ac577591f53bd9552210b1062c6230030be.tar.gz gdb-e54e9ac577591f53bd9552210b1062c6230030be.tar.bz2 |
bundle_lock message tidy
I'd edited these thinking that there might be cases where the counts
were one, but on further investigation it appears not. What's left
here are some minor tweaks.
* read.c (assemble_one, s_bundle_unlock): Formatting.
Consistently add comma and "bytes" to error message.
* testsuite/gas/i386/bundle-bad.l: Adjust to suit.
Diffstat (limited to 'gas/read.c')
-rw-r--r-- | gas/read.c | 21 |
1 files changed, 11 insertions, 10 deletions
@@ -713,19 +713,19 @@ assemble_one (char *line) /* Make sure this hasn't pushed the locked sequence past the bundle size. */ unsigned int bundle_size = pending_bundle_size (bundle_lock_frag); - if (bundle_size > (1U << bundle_align_p2)) - as_bad (_("\ -.bundle_lock sequence at %u bytes but .bundle_align_mode limit is %u bytes"), + if (bundle_size > 1U << bundle_align_p2) + as_bad (_ (".bundle_lock sequence at %u bytes, " + "but .bundle_align_mode limit is %u bytes"), bundle_size, 1U << bundle_align_p2); } else if (bundle_align_p2 > 0) { unsigned int insn_size = pending_bundle_size (insn_start_frag); - if (insn_size > (1U << bundle_align_p2)) - as_bad (_("\ -single instruction is %u bytes long but .bundle_align_mode limit is %u"), - (unsigned int) insn_size, 1U << bundle_align_p2); + if (insn_size > 1U << bundle_align_p2) + as_bad (_("single instruction is %u bytes long, " + "but .bundle_align_mode limit is %u bytes"), + insn_size, 1U << bundle_align_p2); finish_bundle (insn_start_frag, insn_size); } @@ -6244,9 +6244,10 @@ s_bundle_unlock (int arg ATTRIBUTE_UNUSED) size = pending_bundle_size (bundle_lock_frag); - if (size > (1U << bundle_align_p2)) - as_bad (_(".bundle_lock sequence is %u bytes, but bundle size only %u"), - size, 1 << bundle_align_p2); + if (size > 1U << bundle_align_p2) + as_bad (_(".bundle_lock sequence is %u bytes, " + "but bundle size is only %u bytes"), + size, 1u << bundle_align_p2); else finish_bundle (bundle_lock_frag, size); |