This is an archive of the discontinued LLVM Phabricator instance.

[LLVM-C] Improve Bindings For Aliases
ClosedPublic

Authored by CodaFi on May 13 2018, 12:08 PM.

Details

Summary

Add wrappers for a module's alias iterators and a getter and setter for the aliasee value.

Diff Detail

Repository
rL LLVM

Event Timeline

CodaFi created this revision.May 13 2018, 12:08 PM
whitequark requested changes to this revision.May 13 2018, 6:54 PM
whitequark added inline comments.
include/llvm-c/Core.h
2013 ↗(On Diff #146521)

Needs a size argument.

This revision now requires changes to proceed.May 13 2018, 6:54 PM
CodaFi added inline comments.May 13 2018, 10:50 PM
include/llvm-c/Core.h
2013 ↗(On Diff #146521)

I'm going to offer a revision that adds a variant of LLVMValueGetName that returns the length as well to support this then.

CodaFi updated this revision to Diff 147714.May 20 2018, 10:31 AM
CodaFi marked 2 inline comments as done.
whitequark accepted this revision.May 20 2018, 12:04 PM

Thanks!

This revision is now accepted and ready to land.May 20 2018, 12:04 PM
This revision was automatically updated to reflect the committed changes.
Wallbraker added inline comments.
llvm/trunk/lib/IR/Core.cpp
1970

This doesn't use the NameLen argument,

Herald added a project: Restricted Project. · View Herald TranscriptJul 27 2021, 4:04 PM