diff options
author | Fangrui Song <maskray@google.com> | 2018-12-21 22:40:10 +0000 |
---|---|---|
committer | Fangrui Song <maskray@google.com> | 2018-12-21 22:40:10 +0000 |
commit | f874871c9ef28ca044dbbdb92c5a3c35ba8063ae (patch) | |
tree | e1c2810d53af34dc9c388dc971a3d7599dfd91c9 | |
parent | c3f30a7fc6a166910d4b674301e9108f537daf3c (diff) | |
download | bcm5719-llvm-f874871c9ef28ca044dbbdb92c5a3c35ba8063ae.tar.gz bcm5719-llvm-f874871c9ef28ca044dbbdb92c5a3c35ba8063ae.zip |
key method -> key function
The latter is what is actually called in the ABI http://itanium-cxx-abi.github.io/cxx-abi/abi.html#vague-vtable
Pointed out by rsmith
llvm-svn: 349969
-rw-r--r-- | lld/docs/Readers.rst | 2 | ||||
-rw-r--r-- | lld/docs/missingkeymethod.rst | 20 |
2 files changed, 11 insertions, 11 deletions
diff --git a/lld/docs/Readers.rst b/lld/docs/Readers.rst index b69a0b34fab..eae1717f6e5 100644 --- a/lld/docs/Readers.rst +++ b/lld/docs/Readers.rst @@ -74,7 +74,7 @@ files in parallel. Therefore, there should be no parsing state in you Reader object. Any parsing state should be in ivars of your File subclass or in some temporary object. -The key method to implement in a reader is:: +The key function to implement in a reader is:: virtual error_code loadFile(LinkerInput &input, std::vector<std::unique_ptr<File>> &result); diff --git a/lld/docs/missingkeymethod.rst b/lld/docs/missingkeymethod.rst index 3958cc1359c..7e63a4008b2 100644 --- a/lld/docs/missingkeymethod.rst +++ b/lld/docs/missingkeymethod.rst @@ -6,12 +6,12 @@ If your build failed with a linker error something like this:: foo.cc:28: error: undefined reference to 'vtable for C' the vtable symbol may be undefined because the class is missing its key function (see https://lld.llvm.org/missingkeymethod) -it's likely that your class C has a key method (defined by the ABI as the first +it's likely that your class C has a key function (defined by the ABI as the first non-pure, non-inline, virtual method), but you haven't actually defined it. -When a class has a key method, the compiler emits the vtable (and some other -things as well) only in the translation unit that defines that key method. Thus, -if you're missing the key method, you'll also be missing the vtable. If no other +When a class has a key function, the compiler emits the vtable (and some other +things as well) only in the translation unit that defines that key function. Thus, +if you're missing the key function, you'll also be missing the vtable. If no other function calls your missing method, you won't see any undefined reference errors for it, but you will see undefined references to the vtable symbol. @@ -20,7 +20,7 @@ method, and the compiler is forced to emit the vtable in every translation unit that references the class. In this case, it is emitted in a COMDAT section, which allows the linker to eliminate all duplicate copies. This is still wasteful in terms of object file size and link time, so it's always advisable to -ensure there is at least one eligible method that can serve as the key method. +ensure there is at least one eligible method that can serve as the key function. Here are the most common mistakes that lead to this error: @@ -42,8 +42,8 @@ not emit the vtable for ``B``, and you'll get an undefined reference to "vtable for B". This is just an example of the more general mistake of forgetting to define the -key method, but it's quite common because virtual destructors are likely to be -the first eligible key method and it's easy to forget to implement them. It's +key function, but it's quite common because virtual destructors are likely to be +the first eligible key function and it's easy to forget to implement them. It's also more likely that you won't have any direct references to the destructor, so you won't see any undefined reference errors that point directly to the problem. @@ -65,7 +65,7 @@ Say you have an abstract base class declared in a header file:: }; This base class is intended to be abstract, but you forgot to mark one of the -methods pure. Here, ``A::bar``, being non-pure, is nominated as the key method, +methods pure. Here, ``A::bar``, being non-pure, is nominated as the key function, and as a result, the vtable for ``A`` is not emitted, because the compiler is waiting for a translation unit that defines ``A::bar``. @@ -75,10 +75,10 @@ The solution in this case is to add the missing ``= 0`` to the declaration of Key method is defined, but the linker doesn't see it ---------------------------------------------------- -It's also possible that you have defined the key method somewhere, but the +It's also possible that you have defined the key function somewhere, but the object file containing the definition of that method isn't being linked into your application. The solution in this case is to check your dependencies to make sure that -the object file or the library file containing the key method is given to +the object file or the library file containing the key function is given to the linker. |