This is an archive of the discontinued LLVM Phabricator instance.

[msan] Use no-origin functions when possible
ClosedPublic

Authored by vitalybuka on Aug 31 2022, 6:06 PM.

Details

Summary

Saves 1.8% of .text size on CTMark

Diff Detail

Event Timeline

vitalybuka created this revision.Aug 31 2022, 6:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 31 2022, 6:06 PM
vitalybuka requested review of this revision.Aug 31 2022, 6:06 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 31 2022, 6:06 PM
Herald added subscribers: llvm-commits, Restricted Project. · View Herald Transcript
kda accepted this revision.Sep 1 2022, 11:45 AM
This revision is now accepted and ready to land.Sep 1 2022, 11:45 AM

Looks like in all test msan_warning_with_origin_noreturn was renamed to msan_warning_noreturn. Do you have a test that checks that __msan_warning_with_origin_noreturn is emitted?

vitalybuka edited the summary of this revision. (Show Details)Sep 1 2022, 6:14 PM
vitalybuka updated this revision to Diff 457474.Sep 1 2022, 7:09 PM

origins tests

Looks like in all test msan_warning_with_origin_noreturn was renamed to msan_warning_noreturn. Do you have a test that checks that __msan_warning_with_origin_noreturn is emitted?

Done

This revision was landed with ongoing or failed builds.Sep 1 2022, 7:18 PM
This revision was automatically updated to reflect the committed changes.