[LLVMbugs] [Bug 10188] New: FPPassManager::runOnModule may return incorrect values

bugzilla-daemon at llvm.org bugzilla-daemon at llvm.org
Fri Jun 24 16:26:22 PDT 2011


           Summary: FPPassManager::runOnModule may return incorrect values
           Product: libraries
           Version: trunk
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: Core LLVM classes
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: wujingyue at gmail.com
                CC: llvmbugs at cs.uiuc.edu


In lib/VMCore/PassManager, around Line 1530: 

1531 bool FPPassManager::runOnModule(Module &M) {
1532   bool Changed = doInitialization(M);
1534   for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I)
1535     runOnFunction(*I);
1537   return doFinalization(M) || Changed;
1538 }

I think
should be changed to
"Changed |= runOnFunction(*I)"
because FPPassManager::runOnModule is supposed to return true if any function
gets changed. 

Let me know if I'm understanding it correctly. Thanks much. 


Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

More information about the llvm-bugs mailing list