This is an archive of the discontinued LLVM Phabricator instance.

AttributorAttributes: guard against TLI being nullptr
ClosedPublic

Authored by durin42 on Apr 1 2022, 9:45 AM.

Details

Summary

I didn't dig into this very much because it appears to be totally valid
(especially once these properties can come from attributes instead
of only from hard-coded library functions) for TLI to not be defined,
and nothing broke when I added this check, including with all my other
patches applied.

Diff Detail

Unit TestsFailed

Event Timeline

durin42 created this revision.Apr 1 2022, 9:45 AM
Herald added a project: Restricted Project. · View Herald Transcript
durin42 requested review of this revision.Apr 1 2022, 9:45 AM
Herald added a reviewer: sstefan1. · View Herald Transcript
Herald added a reviewer: baziotis. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
This revision is now accepted and ready to land.Apr 1 2022, 11:29 AM
This revision was landed with ongoing or failed builds.Apr 3 2022, 8:19 PM
This revision was automatically updated to reflect the committed changes.