[LLVMbugs] [Bug 23249] New: LCSSA::verifyAnalysis() and verifyAnalysis() does nothing?

[bugzilla-daemon at llvm.org]

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

            Bug ID: 23249
           Summary: LCSSA::verifyAnalysis() and verifyAnalysis() does
                    nothing?
           Product: new-bugs
           Version: trunk
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: normal
          Priority: P
         Component: new bugs
          Assignee: unassignedbugs at nondot.org
          Reporter: yaron.keren at gmail.com
                CC: llvmbugs at cs.uiuc.edu
    Classification: Unclassified

Hi Chandler,

Warnings level 4 were recently enabled for Visual C++ and it complains that:

lib\transforms\utils\lcssa.cpp(345): warning C4718: 'verifyLoop' : recursive
call has no side effects, deleting

essentially, it says that verifyLoop does nothing other than call itself, which
appear to be true both in debug and release modes:

static void verifyLoop(Loop &L, DominatorTree &DT) {
  // Recurse depth-first through inner loops.
  for (Loop::iterator LI = L.begin(), LE = L.end(); LI != LE; ++LI)
    verifyLoop(**LI, DT);

  // Check the special guarantees that LCSSA makes.
  //assert(L.isLCSSAForm(DT) && "LCSSA form not preserved!");
}

-- 
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/20150416/2921373a/attachment.html>