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)