This is an archive of the discontinued LLVM Phabricator instance.

[GlobalIsel] Fix fallback if stack protector isn't supported.
ClosedPublic

Authored by efriedma on May 13 2022, 1:43 PM.

Details

Summary

When GlobalISel fails, we need to report the error, and we need to set the FailedISel property. We skipped those steps if stack protector insertion failed, which led to a very strange miscompile.

Diff Detail

Event Timeline

efriedma created this revision.May 13 2022, 1:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2022, 1:43 PM
efriedma requested review of this revision.May 13 2022, 1:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2022, 1:43 PM
paquette accepted this revision.May 13 2022, 1:51 PM

LGTM, thanks!

This revision is now accepted and ready to land.May 13 2022, 1:51 PM
This revision was landed with ongoing or failed builds.May 13 2022, 2:17 PM
This revision was automatically updated to reflect the committed changes.