Bug 11895 – SimplifyLibCalls should be merged into instcombine

Bug 11895 - SimplifyLibCalls should be merged into instcombine

Summary:

SimplifyLibCalls should be merged into instcombine

Status:	NEW

Product:	libraries
Classification:	Unclassified
Component:	Scalar Optimizations
Version:	trunk
Platform:	PC All

Importance:	P enhancement
Assigned To:	Unassigned LLVM Bugs

URL:
Keywords:	code-cleanup

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2012-01-31 05:26 CST by Chris Lattner
Modified:	2012-02-03 23:04 CST (History)
CC List:	7 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Chris Lattner 2012-01-31 05:26:42 CST

Simplify libcalls was previously split out of instcombine so that it could be
turned off with -fno-builtin.  Now that we have TargetLibraryInfo around, we
don't need to do this.  Merging it into Instcombine's visitCall optimization
logic would eliminate some phase ordering problems that it can have, and is a
generally good cleanup anyway.

Comment 1 Meador Inge 2012-02-01 21:44:03 CST

I would like to pick up this one if nobody is already working on it.

Comment 2 Chris Lattner 2012-02-01 23:47:58 CST

Nice, please propose a plan on llvmdev, incremental is good :)

Comment 3 Meador Inge 2012-02-03 23:04:39 CST

Cool.  I am working on a proposal now.