This is an archive of the discontinued LLVM Phabricator instance.

[ORC] Use ExecutorAddr::toPtr() in documentation
ClosedPublic

Authored by vadorovsky on May 12 2023, 9:55 AM.

Details

Summary

Partial move from JITTargetAddress to ExecutorAddr was done in 8b1771bd9f30,
but that change missed the ORC and Kaleidoscope documents which led to
their inconsistency.

Fixes #62411

Diff Detail

Event Timeline

vadorovsky created this revision.May 12 2023, 9:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 12 2023, 9:55 AM
vadorovsky requested review of this revision.May 12 2023, 9:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 12 2023, 9:55 AM
lhames accepted this revision.May 12 2023, 7:43 PM
lhames added a subscriber: lhames.

Looks good to me. Thanks @vadorovsky!

Do you have commit access? If not, please let me know what name and email address you would like used for attribution and I can commit on your behalf.

This revision is now accepted and ready to land.May 12 2023, 7:43 PM

@lhames I don't have commit access (it's my first contribution ever). You can use

Mike Rostecki <vadorovsky@protonmail.com>

for attribution. Thanks!

I see that CI is red (unrelated to this patch). Is there some way to re-trigger the build, or is it fine?

This revision was landed with ongoing or failed builds.May 31 2023, 3:16 PM
This revision was automatically updated to reflect the committed changes.