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.

Sign up for GitHub

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

Jump to bottom

Backport 6c99e631201a to 12.0.1 to fix SCEV issue causing a mis-compile #49637

Closed
fhahn opened this issue May 11, 2021 · 3 comments
Closed

Backport 6c99e631201a to 12.0.1 to fix SCEV issue causing a mis-compile #49637

fhahn opened this issue May 11, 2021 · 3 comments
Labels
bugzilla Issues migrated from bugzilla loopoptim

Comments

@fhahn
Copy link
Contributor

fhahn commented May 11, 2021

Bugzilla Link 50293
Resolution FIXED
Resolved on May 19, 2021 20:04
Version trunk
OS All
Blocks #48661
CC @tstellar
Fixed by commit(s) d65e5f6 6c99e63 0ef7836 4e46ff4

Extended Description

It would be great if we could pick d65e5f6 (adds tests) and 6c99e63 (fix) onto the 12.0.1 branch.

It fixes a mis-compile as illustrated in the changed test, which users can hit with relatively simple C code.
@tstellar
Copy link
Collaborator

The fix does not apply cleanly, could someone backport this and push a branch to their local github fork?

@fhahn
Copy link
Contributor Author

fhahn commented May 13, 2021

The fix does not apply cleanly, could someone backport this and push a
branch to their local github fork?

It looks like the fixes have been applied to 12.x as
0ef7836
4e46ff4

Please let me know if there's anything else left to do.

@tstellar
Copy link
Collaborator

Merged: 4e46ff4

@tstellar tstellar mentioned this issue Feb 22, 2021
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla loopoptim
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tstellar @fhahn