On Sun, 2013-07-07 at 19:54 +0200, Georg-Johann Lay wrote:
> Ondrej Bilka schrieb:
> 
> > http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
> 

Below are some other hunks that look suspicious...
(trying not to duplicate the things already mentioned by others)

- * 1) It means that finalizers, and all methods calle by them,
+ * 1) It means that finalizers, and all methods callee by them,

-> 'called'


-      /*  SET_ACCESS, we want to set an explicte set of permissions, do not
+      /*  SET_ACCESS, we want to set an explicate set of permissions, do not

-> 'explicit'


    In Objective-C, there are two additional variants:
 
    foreach-statement:
-     for ( expression in expresssion ) statement
+     for ( expression in expressions ) statement
      for ( declaration in expression ) statement

Really?  I'm not so sure.


-   configury */
+   configure */

-> 'configury'


-       calll    #gettlsoff(ADDR)@(gr8, gr0)
+       call    #gettlsoff(ADDR)@(gr8, gr0)


-       calll   #gettlsoff(ADDR)@(gr8, gr0)
+       call   #gettlsoff(ADDR)@(gr8, gr0)

The original 'calll' is correct (see frv.md).


-   have the same priority - candidate is best if its dependees were
+   have the same priority - candidate is best if its dependencies were

-> 'dependees'


-   does not look at other present displacement addressings around it.
+   does not look at other present displacement addressing around it.

-> 'addressings' (as in addressing modes)


-  CM_SMALL,    /* Makes various assumpation about sizes of code and
+  CM_SMALL,    /* Makes various assumption about sizes of code and

-  CM_SMALL_PIC,        /* Makes various assumpation about sizes of code and
+  CM_SMALL_PIC,        /* Makes various assumption about sizes of code and

-> 'assumptions'


           /* If we had fewer function args than explicit template args,
-             just use the explicits.  */
+             just use the explicit.  */

-> 'explicit ones'


-            array reference if the where and elswhere destinations
+            array reference if the where and Elsinore destinations

-> 'elsewhere'


-   We provide accestor to the inline_summary datastructure and
+   We provide accessor to the inline_summary datastructure and

-> probably 'ancestor'


-/* The array used to find duplications in conflict vectors of
+/* The array used to find duplication in conflict vectors of

-/* Remove duplications in conflict vector of OBJ.  */
+/* Remove duplication in conflict vector of OBJ.  */

-/* Process operator duplications in insn with ID.  We do it after the
+/* Process operator duplication in insn with ID.  We do it after the

-> 'duplicates' maybe?


-   function we iterate decompressions until no data remains.  */
+   function we iterate decompression's until no data remains.  */

-> 'decompressions'


-   TODO: Make into some kind of configury-generated table.  */
+   TODO: Make into some kind of configure-generated table.  */

-> 'configury-generated'


-        point of view as prefetch withouth dependecies will have a
+        point of view as prefetch withouth dependencies will have a

-> missed 'without'


-   * Unique vinsn derivates from CALL, ASM, JUMP (for a while) and other
+   * Unique vinsn derivatives from CALL, ASM, JUMP (for a while) and other

-> maybe 'deviates' ?


-/* Find the set of registers that are unavailable for storing expres
+/* Find the set of registers that are unavailable for storing express

-   that are not available for storing expres while moving ORIG_OPS up on the
+   that are not available for storing express while moving ORIG_OPS up on the

-      /* Merge c_expres found or unify live register sets from different
+      /* Merge c_express found or unify live register sets from different


-> maybe 'expression' ?


-/* { dg-final { scan-assembler "calll.*#gettlsoff\\(0\\)" } } */
+/* { dg-final { scan-assembler "call.*#gettlsoff\\(0\\)" } } */

-> see above for the 'calll' frv case.  This breaks the test case.
Do not change things inside /* { } */ comments in test cases.


-/* PR target/50749: Verify that subsequent post-increment addressings
+/* PR target/50749: Verify that subsequent post-increment addressing

-/* PR target/50749: Verify that subsequent pre-decrement addressings
+/* PR target/50749: Verify that subsequent pre-decrement addressing

-/* PR target/50749: Verify that subsequent post-increment addressings
+/* PR target/50749: Verify that subsequent post-increment addressing

-/* PR target/50749: Verify that subsequent pre-decrement addressings
+/* PR target/50749: Verify that subsequent pre-decrement addressing

-> 'addressings' (as in addressing modes)


-       /* For buitins that are likely expanded to nothing or
+       /* For builtin's that are likely expanded to nothing or

-> 'builtins'


    BASE must be either a declaration or a memory reference that has correct
-   alignment ifformation embeded in it (e.g. a pre-existing one in SRA).  */
+   alignment ifformation embedded in it (e.g. a pre-existing one in SRA).  */

-> missed 'information' I guess...


-     via commutating the addition and contracting operations to zero
+     via commuting the addition and contracting operations to zero

-> 'commutating'


-        and one can be commutated try that for the scalar stmts
+        and one can be commuted try that for the scalar stmts

-> 'commutated'


-   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algoritms
+   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algorithms

both links do 404 anyway ;)


-   Written by Cary Coutant <ccout...@google.com>.
+   Written by Cary Coutant <account...@google.com>.

As Georg-Johann mentioned, do not touch email addresses.


-  /* This definition was pre-empted by a definition in a regular
+  /* This definition was pre-emptied by a definition in a regular

-  /* This definition was pre-empted by a definition in another IR file.  */
+  /* This definition was pre-emptied by a definition in another IR file.  */

-> 'preempted' ?


-  set it true anyway, since malloc copes with non-contiguities.
+  set it true anyway, since malloc copes with non-continuities.

-> 'non-contiguities'


-/* mstate bit set if continguous morecore disabled or failed */
+/* mstate bit set if continuous morecore disabled or failed */

-> 'contiguous'


-/* Chebyshev polynom coeficients for (exp(x)-1)/x */
+/* Chebyshev polynom coefficients for (exp(x)-1)/x */

-> 'polynomial'


-  // Randomize the colour, just for asthetics =)
+  // Randomize the colour, just for aesthetics =)

-> missed 'color' (in a couple of places actually)


Reply via email to