Commit: Fix compile time warning in x86-64-level.sh

Nick Clifton nickc@redhat.com
Tue Nov 3 08:56:08 GMT 2020


Hi Guys,

  I am applying the patch below to fix this compile time warning when
  building the BFD linker with clang:

    eelf_i386.c:5069:6: error: misleading indentation; statement is not part of the previous 'if' [-Werror,-Wmisleading-indentation]
            params.isa_level = level;
            ^
    eelf_i386.c:5067:4: note: previous statement is here
          if (*end != 0 || level < 2 || level > 4)
          ^

Cheers
  Nick

ld/ChangeLog
2020-11-03  Nick Clifton  <nickc@redhat.com>

	* emulparams/x86-64-level.sh
	(PARSE_AND_LIST_ARGS_CASE_Z_X86_64_LEVEL): Fix indentation.

diff --git a/ld/emulparams/x86-64-level.sh b/ld/emulparams/x86-64-level.sh
index c74b236900..c46aacf3af 100644
--- a/ld/emulparams/x86-64-level.sh
+++ b/ld/emulparams/x86-64-level.sh
@@ -11,7 +11,7 @@ PARSE_AND_LIST_ARGS_CASE_Z_X86_64_LEVEL='
          unsigned int level = strtoul (optarg + 8 , &end, 10);
          if (*end != '\0' || level < 2 || level > 4)
            einfo (_("%F%P: invalid x86-64 ISA level: %s\n"), optarg);
-           params.isa_level = level;
+         params.isa_level = level;
        }
 '
 



More information about the Binutils mailing list