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

Uninitialized memory treated as poison #1067

Open
nikic opened this issue Jul 3, 2024 · 2 comments
Open

Uninitialized memory treated as poison #1067

nikic opened this issue Jul 3, 2024 · 2 comments

Comments

@nikic
Copy link
Contributor

nikic commented Jul 3, 2024

It looks like alive2 is treating uninitialized memory as poison: https://alive2.llvm.org/ce/z/VKRwvf

define i8 @src() {
  %a = alloca i8, align 1
  %v = load i8, ptr %a, align 1
  ret i8 %v
}

define i8 @tgt() {
  ret i8 poison
}

This doesn't match LLVM's current semantics.

@IamYJLee
Copy link
Contributor

IamYJLee commented Jul 9, 2024

@nikic @nunoplopes
I would like to try to fix this problem. Could I get some help to do so?

@nunoplopes
Copy link
Member

First of all, it's not an easy fix.
Secondly, I'm trying to fix LLVM rather than fixing Alive2 🙂 Hopefully we can fix LLVM instead in the next few months.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants