Page MenuHomePhabricator

[IR][FIX] Intrinsics - don't apply default willreturn if IntrNoReturn is specified
ClosedPublic

Authored by sstefan1 on Thu, Oct 1, 2:35 AM.

Details

Summary

Since willreturn will soon be added as default attribute (D86021), we can end up with both noreturn and willreturn on the same intrinsic. This was exposed by llvm.wasm.throw which has IntrNoReturn.

Diff Detail

Event Timeline

sstefan1 created this revision.Thu, Oct 1, 2:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Oct 1, 2:35 AM
sstefan1 requested review of this revision.Thu, Oct 1, 2:35 AM
sstefan1 edited the summary of this revision. (Show Details)Thu, Oct 1, 2:40 AM
arsenm accepted this revision.Thu, Oct 1, 6:55 AM
This revision is now accepted and ready to land.Thu, Oct 1, 6:55 AM
This revision was landed with ongoing or failed builds.Mon, Oct 12, 2:30 AM
This revision was automatically updated to reflect the committed changes.