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

SCCP miscompiles sitofp(undef) #7312

Closed
sunfishcode opened this issue Apr 26, 2010 · 1 comment
Closed

SCCP miscompiles sitofp(undef) #7312

sunfishcode opened this issue Apr 26, 2010 · 1 comment
Labels
bugzilla Issues migrated from bugzilla

Comments

@sunfishcode
Copy link
Member

Bugzilla Link 6940
Resolution FIXED
Resolved on Apr 26, 2010 13:21
Version trunk
OS All

Extended Description

On this testcase:

define double @​foo() {
%t = sitofp i32 undef to double
ret double %t
}

SCCP changes the sitofp operand to undef. This is over-aggressive, since there exist double values which sitofp cannot produce. For comparison, instcombine on this testcase changes the operand to 0.0.

@lattner
Copy link
Collaborator

lattner commented Apr 26, 2010

Fixed in r102358, thanks.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
cyndyishida pushed a commit to cyndyishida/llvm-project that referenced this issue Sep 5, 2023
…-20230725

🍒[APINotes] Support globals in `extern "C++"` blocks
banach-space added a commit to banach-space/llvm-project that referenced this issue Dec 14, 2023
Direct follow-up of llvm#7312 - the linker on Darwin does not support
`-whole-archive`, so that needs to be removed from the linker
invocation.

For context:
  * llvm#7312
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
Projects
None yet
Development

No branches or pull requests

2 participants