-
-
Notifications
You must be signed in to change notification settings - Fork 17.8k
ScalarPair product types missing noundef in LLVM return types #157589
Copy link
Copy link
Open
Labels
A-LLVMArea: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.C-enhancementCategory: An issue proposing an enhancement or a PR with one.Category: An issue proposing an enhancement or a PR with one.needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triagingThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
Description
Activity
Metadata
Metadata
Assignees
Labels
A-LLVMArea: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.C-enhancementCategory: An issue proposing an enhancement or a PR with one.Category: An issue proposing an enhancement or a PR with one.needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triagingThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
We do set
noundefon scalar return types, but for a pair like(u16, bool)-- which can't have any undef in the LLVM immediate pair type even though it can in memory -- we're currently not:We just generate https://rust.godbolt.org/z/W154v8xhW
It should be
If it's easier, it looks like we're not setting it for things that don't have undef in memory either (https://rust.godbolt.org/z/rez44Kx58) so could start with that part.