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?