Emit all result types of a multivalue func type
Details
Diff Detail
Event Timeline
Nope, I guess we don't have any tests for this yet. @sammax, could you add a test in lld/test/wasm?
For feature testing, it would probably be cleanest to check all the input functions in Writer::populateTargetFeatures, since that's where all the other feature checking happens. @sammax, would you want to add that functionality as well? If not, it would be fine to leave that to a separate patch, I think.
@tlively regarding the feature testing, I don't quite understand what happens in Writer::populateTargetFeatures yet, I'll have to look into it a bit more.
@sammax Yeah there's a lot of stuff going on there. Thinking about it more, I'm not sure we should be feature checking type signatures in the linker. We certainly aren't feature checking the contents of the linked functions. This patch LGTM, so I'll go ahead and land it as-is.