[LLVMbugs] [Bug 19402] New: Finish converting instcombine to use IRBuilder

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=19402

            Bug ID: 19402
           Summary: Finish converting instcombine to use IRBuilder
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: Scalar Optimizations
          Assignee: unassignedbugs at nondot.org
          Reporter: rafael.espindola at gmail.com
                CC: bigcheesegs at gmail.com, filcab at gmail.com,
                    llvmbugs at cs.uiuc.edu
    Classification: Unclassified

Instcombine got an IR builder in r80492, but the move is still not done. Right
now a we have a mix of visit methods doing

Foo = Builder->CreateFoo()...
return ReplaceInstUsesWith(I, Foo);

or just

return new Foo(...);

and letting the main loop insert the instruction in the BB and replace it.


Using the builder looks like the new way, but it seems to be missing some
features, like reusing the old instruction name.

-- 
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/20140411/302c7d6e/attachment.html>