This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
libgcc: Fix typos in comments for ARM FP emulation routines
- From: Martin Galvan <martin dot galvan at tallertechnologies dot com>
- To: gcc-patches at gcc dot gnu dot org, ramana dot gcc at googlemail dot com
- Date: Wed, 13 Apr 2016 17:41:28 -0300
- Subject: libgcc: Fix typos in comments for ARM FP emulation routines
- Authentication-results: sourceware.org; auth=none
A lifetime ago I contributed a patch that added CFI directives to ieee754-df.S,
among other files. For unrelated reasons I looked at that file again and saw
that some of the comments have extra '@' characters interwined; this is probably
the result of splitting lines because they were too long. This patch simply
removes those extra chars, as well as fixing a couple other cosmetic issues.
libgcc/ChangeLog:
2016-04-13 Martin Galvan <martin.galvan@tallertechnologies.com>
* config/arm/ieee754-df.S: Fix typos in comments.
Index: libgcc/config/arm/ieee754-df.S
===================================================================
--- libgcc/config/arm/ieee754-df.S (revision 234960)
+++ libgcc/config/arm/ieee754-df.S (working copy)
@@ -160,8 +160,8 @@
teq r4, r5
beq LSYM(Lad_d)
-@ CFI note: we're lucky that the branches to Lad_* that appear after this function
-@ have a CFI state that's exactly the same as the one we're in at this
+@ CFI note: we're lucky that the branches to Lad_* that appear after this
+@ function have a CFI state that's exactly the same as the one we're in at this
@ point. Otherwise the CFI would change to a different state after the branch,
@ which would be disastrous for backtracing.
LSYM(Lad_x):
@@ -1158,8 +1158,8 @@
1: str ip, [sp, #-4]!
.cfi_adjust_cfa_offset 4 @ CFA is now sp + previousOffset + 4.
@ We're not adding CFI for ip as it's pushed into the stack
- @ only because @ it may be popped off later as a return value
- @ (i.e. we're not preserving @ it anyways).
+ @ only because it may be popped off later as a return value
+ @ (i.e. we're not preserving it anyways).
@ Trap any INF/NAN first.
mov ip, xh, lsl #1
@@ -1169,14 +1169,14 @@
COND(mvn,s,ne) ip, ip, asr #21
beq 3f
.cfi_remember_state
- @ Save the current CFI state. This is done because the branch
- @ is conditional, @ and if we don't take it we'll issue a
- @ .cfi_adjust_cfa_offset and return. @ If we do take it,
- @ however, the .cfi_adjust_cfa_offset from the non-branch @ code
- @ will affect the branch code as well. To avoid this we'll
- @ restore @ the current state before executing the branch code.
+ @ Save the current CFI state. This is done because the branch
+ @ is conditional, and if we don't take it we'll issue a
+ @ .cfi_adjust_cfa_offset and return. If we do take it,
+ @ however, the .cfi_adjust_cfa_offset from the non-branch code
+ @ will affect the branch code as well. To avoid this we'll
+ @ restore the current state before executing the branch code.
- @ Test for equality. @ Note that 0.0 is equal to -0.0.
+ @ Test for equality. Note that 0.0 is equal to -0.0.
2: add sp, sp, #4
.cfi_adjust_cfa_offset -4 @ CFA is now sp + previousOffset.