[llvm-bugs] [Bug 28403] New: warning:=?UTF-8?Q?=20this=20=E2=80=98if=E2=80=99=20clause=20does=20not=20guard=2E=2E=2E=20?=[-Wmisleading-indentation]

[via llvm-bugs]

https://llvm.org/bugs/show_bug.cgi?id=28403

            Bug ID: 28403
           Summary: warning: this ‘if’ clause does not guard...
                    [-Wmisleading-indentation]
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: LLVM assembly language parser
          Assignee: unassignedbugs at nondot.org
          Reporter: db48x at db48x.net
                CC: llvm-bugs at lists.llvm.org
    Classification: Unclassified

While building emscripten, which uses a fork of llvm, I happened to notice the
following warning scroll by:

/home/db48x/src/emsdk_portable/clang/fastcomp/src/lib/MC/MCParser/DarwinAsmParser.cpp:
In member function ‘bool {anonymous}::DarwinAsmParser::parseVersionMin(
llvm::StringRef, llvm::SMLoc)’:
/home/db48x/src/emsdk_portable/clang/fastcomp/src/lib/MC/MCParser/DarwinAsmParser.cpp:932:3:
warning: this ‘if’ clause does not guard... [-Wmisleading-indenta
tion]
   if (Update > 255 || Update < 0)
   ^~
/home/db48x/src/emsdk_portable/clang/fastcomp/src/lib/MC/MCParser/DarwinAsmParser.cpp:934:5:
note: ...this statement, but the latter is misleadingly indented 
as if it is guarded by the ‘if’
     Lex();
     ^~~

In the llvm svn repository it's on a slightly different line,
DarwinAsmParser.cpp:962.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20160703/d53b6ca8/attachment-0001.html>