This is an archive of the discontinued LLVM Phabricator instance.

Expose `DIBuilder::finalizeSubprogram()` through the LLVM C API
ClosedPublic

Authored by koutheir on Jun 23 2021, 10:06 AM.

Details

Summary

Expose DIBuilder::finalizeSubprogram() through the LLVM C API.

The LLVM C API function is called LLVMDIBuilderFinalizeSubprogram().

Diff Detail

Event Timeline

koutheir created this revision.Jun 23 2021, 10:06 AM
koutheir requested review of this revision.Jun 23 2021, 10:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 23 2021, 10:06 AM
koutheir retitled this revision from Expose `DIBuilder::finalizeSubprogram()` through the LLVM C API. The LLVM C API function is called `LLVMDIBuilderFinalizeSubprogram()`. to Expose `DIBuilder::finalizeSubprogram()` through the LLVM C API.Jun 23 2021, 10:06 AM
koutheir edited the summary of this revision. (Show Details)

The build failures seem unrelated to this patch. Can someone review, please?

lattner resigned from this revision.Jun 24 2021, 12:22 PM
CodaFi accepted this revision.Jun 25 2021, 6:14 AM
CodaFi added a subscriber: CodaFi.

LGTM

Thanks

This revision is now accepted and ready to land.Jun 25 2021, 6:14 AM
koutheir added a comment.EditedJul 7 2021, 4:56 PM

@CodaFi, can you please commit this accepted change?

Is there still an issue preventing committing of this change?

Is this patch uninteresting to commit?

This revision was landed with ongoing or failed builds.Sep 30 2021, 9:00 PM
This revision was automatically updated to reflect the committed changes.

Is this patch uninteresting to commit?

Committed, sorry for the delay!