The langref already states it does, but this wasn't implemented. Also
covers inalloca and preallocated. Also helps fix a dependence on
pointer element types.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM, one minor request.
| llvm/test/Transforms/Attributor/value-simplify.ll | ||
|---|---|---|
| 355 | ups. Could you maybe repair the tests to avoid UB here? | |
| llvm/test/Transforms/Attributor/value-simplify.ll | ||
|---|---|---|
| 355 | I'm not sure what this is trying to test exactly. What argument should this pass instead? | |
Comment Actions
| llvm/test/Transforms/Attributor/value-simplify.ll | ||
|---|---|---|
| 355 | Just forwarding an argument seems to have worked | |
| llvm/test/Transforms/Attributor/value-simplify.ll | ||
|---|---|---|
| 355 | Add an argument and pass it along so this is not UB. The test checks we do not do anything invalid with a complicated argument (here inalloca) | |
ups. Could you maybe repair the tests to avoid UB here?