OSDN Git Service

Remove NameLen argument from newly-introduced IR C APIs.
authorJames Y Knight <jyknight@google.com>
Mon, 14 Jan 2019 17:16:55 +0000 (17:16 +0000)
committerJames Y Knight <jyknight@google.com>
Mon, 14 Jan 2019 17:16:55 +0000 (17:16 +0000)
commit4f613a607d8d19203372de0bd495ade3d9cd59af
treec58da52cf4770f28f936b65cdda28bc312f493a4
parent7cae0753be3bf8bdf753409fe98cf84b6062f386
Remove NameLen argument from newly-introduced IR C APIs.

Normally, changing the function signatures of C APIs is disallowed,
but as these two are brand new last week, and haven't been released
yet, it is okay in this instance.

As per discussion in D56556, we will not add NameLen arguments to IR
building APIs, for the following reasons:

1. We do not want to deprecate all of the IR building APIs, just to add a
NameLen argument to each one.

2. Consistency is important, so adding it just to new ones is unfortunate.

3. The IR names are completely optional, useful for readability of IR
only. There is no value in ever supporting nul bytes.

Differential Revision: https://reviews.llvm.org/D56669

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@351076 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm-c/Core.h
lib/IR/Core.cpp