Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SimplifyLibCalls misses optimization #1687

Closed
llvmbot opened this issue Apr 8, 2007 · 1 comment
Closed

SimplifyLibCalls misses optimization #1687

llvmbot opened this issue Apr 8, 2007 · 1 comment
Labels
bugzilla Issues migrated from bugzilla ipo Interprocedural optimizations

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Apr 8, 2007

Bugzilla Link 1315
Resolution FIXED
Resolved on Feb 22, 2010 12:52
Version trunk
OS All
Reporter LLVM Bugzilla Contributor

Extended Description

After Chris' round of changes to SimplifyLibCalls, it is now missing an
optimization that it previously performed, as identified by this test case
failing:
FAIL: /proj/llvm/llvm-1/test/Transforms/SimplifyLibCalls/SPrintF.ll:
child process exited abnormally
%r4 = call i32 (i8*, i8*, ...)* @​sprintf( i8* %target_p, i8* %fmt1_p, i8*
%hello_p ) ; [#uses=1]
%r4.1 = call i32 (i8*, i8*, ...)* @​sprintf( i8* %target_p, i8* %fmt1_p, i8* %p
) ; [#uses=1]
%r5 = call i32 (i8*, i8*, ...)* @​sprintf( i8* %target_p, i8* %fmt2_p, i32 82 )
; [#uses=1]

Some of these are important for performance. For example, the first one should
be reduced to a few stores.

@lattner
Copy link
Collaborator

lattner commented Apr 8, 2007

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla ipo Interprocedural optimizations
Projects
None yet
Development

No branches or pull requests

2 participants