-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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 it if it has been sufficiently triaged.This issue may need triage. Remove it if it has been sufficiently triaged.
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 it if it has been sufficiently triaged.This issue may need triage. Remove it if it has been sufficiently triaged.
Type
Fields
Give feedbackNo fields configured for issues without a type.
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.