[llvm-commits] First contribution to LLVM: improve the internalize pass

Pierre-Nicolas Clauss pini at tuxfamily.org
Fri Jun 22 14:38:34 PDT 2012


Hi, LLVM folks

I've been working with LLVM for some time now, and I started to feel
the need to make some small improvements here and there.
So, this is my actual first contribution: two changes on the internalize pass.

The first change introduces a new command line option to specify the
public API symbols by a common prefix. A lot of projects are using
this kind of naming convention, so this can help to use internalize in
one pass instead of first listing the public API based on a prefix and
then call internalize on this list.

The second change also introduces a new command line option to choose
whether to use the internal or the private linkage for the symbols
internalized.

I generated the two patches with "git format-patch", as stated in the
"Getting Started" web page, and I hope these changes cope with the
contribution expectations of the team.

Best regards,

-- 
Pierre-Nicolas "pini" Clauss.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Keep-public-symbols-by-prefix.patch
Type: application/octet-stream
Size: 4567 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120622/6b7bbe0a/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Internalize-to-either-private-or-internal-linkage.patch
Type: application/octet-stream
Size: 4447 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120622/6b7bbe0a/attachment-0001.obj>


More information about the llvm-commits mailing list