This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/4] Fix misleading indentation in tree-ssa-loop-unswitch.c


On 10/29/2015 10:49 AM, David Malcolm wrote:
tree-ssa-loop-unswitch.c has this bad indentation at line 452:

    449        if (dump_file && (dump_flags & TDF_DETAILS))
    450          fprintf (dump_file, ";; Not unswitching, loop is not expected"
    451                   " to iterate\n");
    452          return false;

which leads to this warning from -Wmisleading-indentation (justifiably,
in my opinion):

../../../src/gcc/tree-ssa-loop-unswitch.c: In function âbool tree_unswitch_outer_loop(loop*)â:
../../../src/gcc/tree-ssa-loop-unswitch.c:452:2: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]
   return false;
   ^
../../../src/gcc/tree-ssa-loop-unswitch.c:449:7: note: ...this âifâ clause, but it is not
        if (dump_file && (dump_flags & TDF_DETAILS))
        ^

This patch fixes the indentation of the "return false;"

gcc/ChangeLog:
	* tree-ssa-loop-unswitch.c (tree_unswitch_outer_loop): Fix
	indentation.
OK.
jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]