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 <[email protected]>.
+ Written by Cary Coutant <[email protected]>.
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)