This is an archive of the discontinued LLVM Phabricator instance.

[LLVM-C] Use NameLen in LLVMGetNamedGlobalAlias
ClosedPublic

Authored by Wallbraker on Jan 1 2022, 2:00 PM.

Details

Summary

I tried to look over the file and didn't see any other non-use of *Len variables.

Diff Detail

Unit TestsFailed

Event Timeline

Wallbraker created this revision.Jan 1 2022, 2:00 PM
Wallbraker requested review of this revision.Jan 1 2022, 2:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 1 2022, 2:00 PM
deadalnix accepted this revision.Jan 1 2022, 2:05 PM
deadalnix added a subscriber: deadalnix.

Thanks.

This revision is now accepted and ready to land.Jan 1 2022, 2:05 PM

Thank you for the quick review, I don't have commit access so I will need help to have it land.

I have no idea what the build test error is about, looks super unrelated.

I can commit for you, what "Name <email>" would you like?

@aeubanks Thanks: "Jakob Bornecrantz <jakob@collabora.com>"

This revision was landed with ongoing or failed builds.Jan 19 2022, 8:59 AM
This revision was automatically updated to reflect the committed changes.