summaryrefslogtreecommitdiffstats
path: root/lld/lib/ReaderWriter
Commit message (Collapse)AuthorAgeFilesLines
...
* [PECOFF] Do not use anonymous namespace in a header.Rui Ueyama2014-05-201-6/+6
| | | | llvm-svn: 209243
* Fix typosAlp Toker2014-05-202-2/+2
| | | | llvm-svn: 209213
* [Mips] Show warning if the linker cannot find a pair for a R_MIPS_HI16Simon Atanasyan2014-05-191-4/+6
| | | | | | | relocation. In fact this case violates ABI but sometimes compilers might produce such code. llvm-svn: 209153
* [Mips] Fix calculation of initial GOT entry value when this entry isSimon Atanasyan2014-05-171-1/+1
| | | | | | referenced by a local symbol. llvm-svn: 209063
* [mach-o] Add support for zero-fill sections.Nick Kledzik2014-05-154-14/+58
| | | | llvm-svn: 208928
* [mach-o] Add support and test cases for parsing tentative definitionsNick Kledzik2014-05-156-4/+76
| | | | llvm-svn: 208919
* [PECOFF] Read files lazily.Rui Ueyama2014-05-141-9/+20
| | | | | | | | ExportedSymbolRenameFile is not always used. In most cases we don't need to read given files at all. So lazy load would help. This doesn't change the meaining of the program. llvm-svn: 208818
* Add -print_atoms options to DarwinLdDriver which dumps final state of all ↵Nick Kledzik2014-05-142-3/+7
| | | | | | atoms in yaml llvm-svn: 208813
* [PECOFF] Find symbols with @number suffix for dllexported symbolsRui Ueyama2014-05-142-1/+133
| | | | | | | | | | | | | | | | | | | | As written in the comment in this patch, symbol names specified with /export option is resolved in a special way; for /export:foo, linker finds a foo@<number> symbol if such symbols exists. On Windows, a function in stdcall calling convention is mangled with a leading underscore and following "@" and numbers. This name mangling is kind of automatic, so you can sometimes omit _ and @number when specifying a symbol. /export option is that case. Previously, if a file in an archive file foo.lib provides a symbol _fn@8, and /export:fn is specified, LLD failed to resolve the symbol. It only tried to find _fn, and failed to find _fn@8. With this patch, _fn@8 will be searched on the second iteration. Differential Revision: http://reviews.llvm.org/D3736 llvm-svn: 208754
* Add observers to Input GraphRui Ueyama2014-05-141-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | Make it possible to add observers to an Input Graph, so that files returned from an Input Graph can be examined before they are passed to Resolver. To implement some PE/COFF features we need to know all the symbols that *can* be solved, including ones in archive files that are not yet to be read. Currently, Resolver only maintains a set of symbols that are already read. It has no knowledge on symbols in skipped files in an archive file. There are many ways to implement that. I chose to apply the observer pattern here because it seems most non-intrusive. We don't want to mess up Resolver with architecture specific features. Even in PE/COFF, the feature that needs this mechanism is minor. So I chose not to modify Resolver, but add a hook to Input Graph. Differential Revision: http://reviews.llvm.org/D3735 llvm-svn: 208753
* [ELF] Emit DT_TEXTREL dynamic table flag.Simon Atanasyan2014-05-131-0/+16
| | | | | | | | | | | | | | If one or more dynamic relocation might modify a read-only section, dynamic table should contain DT_TEXTREL tag. The patch introduces new `RelocationTable::canModifyReadonlySection()` method. This method checks through the relocations to see if any modifies a read-only section. The DynamicTable class calls this method and emits the DT_TEXTREL tag if necessary. The patch reviewed by Rui Ueyama and Shankar Easwaran. llvm-svn: 208670
* [PECOFF] Resolve dllexported symbols.Rui Ueyama2014-05-091-0/+1
| | | | | | | | | | | | | | | | | We did not actively try to resolve dllexported symbols specified by /export or by a module definition file. So if exported symbols would be resolved for other reasons, like other symbols refer to them, that was fine, but if (unreferenced) exported symbols were in an archive file, and no one refers to that file in the archive, they remained unresolved. That would obviously cause the issue that dllexported symbols are not in a resultant DLL. In this patch, we create an undefined symbol for each dllexported symbol, to let the core linker to resolve it. llvm-svn: 208452
* [PECOFF] Fix dllexported name.Rui Ueyama2014-05-091-1/+8
| | | | | | | | Previously the handling of exported symbol was wrong if it's specified in a module definition file in the form of <externalname>=<internalname>. Export the correct symbol. llvm-svn: 208446
* Revert "temporary commit."Rui Ueyama2014-05-092-71/+0
| | | | | | This reverts accidental commit r208427. llvm-svn: 208433
* temporary commit.Rui Ueyama2014-05-092-0/+71
| | | | llvm-svn: 208427
* [PECOFF] Split LocallyImportedSymbolFile into two classes.Rui Ueyama2014-05-081-29/+36
| | | | | | | I have a plan to use VirtualArchiveFile in this file in the near future. This change should improve the readability by itself, too. llvm-svn: 208365
* Re-submit r207884: Remove dead codeRui Ueyama2014-05-056-19/+0
| | | | | | Differential Revision: http://reviews.llvm.org/D3599 llvm-svn: 207989
* [ELF] Use a range based access to the ELFFile's sections collection.Simon Atanasyan2014-05-052-24/+19
| | | | | | No functional changes. llvm-svn: 207953
* [Mips] R_MIPS_GPREL32 relocation support.Simon Atanasyan2014-05-054-5/+103
| | | | llvm-svn: 207949
* Revert "Remove dead code."Rui Ueyama2014-05-026-0/+19
| | | | | | This reverts commit r207884 which was prematurely committed by accident. llvm-svn: 207886
* Remove "virtual" and add "override".Rui Ueyama2014-05-021-4/+4
| | | | llvm-svn: 207885
* Remove dead code.Rui Ueyama2014-05-026-19/+0
| | | | | | | | | | | | | | | isAlias always returns false and no one is using it. It was originally added Atom to query if an atom is an alias for another atom, assuming that alias atoms are different from normal atoms. We now support atom aliasing, but the way that's implemented is in a different way than what isAlias assumed. An alias atom is just a regular defined atom with no content, and it has a layout- before edge to alias-to atom so that they are layed out at the same location in the result. So this is dead code, and it doesn't make much sense to keep it. llvm-svn: 207884
* [PECOFF] Support =internalName syntax in .def file.Rui Ueyama2014-05-022-2/+3
| | | | | | | | | | | Export definitions in a module definition file is as follows: exportedname[=internalname] [@ordinal [NONAME]] [PRIVATE] [DATA] Previously we did not support =internalname, so users couldn't export symbols from a DLL with a different name. llvm-svn: 207827
* Make the variable scope narrower. No functionality change.Rui Ueyama2014-05-021-5/+2
| | | | llvm-svn: 207821
* [PECOFF] Drop stdcall's atsign suffix only.Rui Ueyama2014-05-011-2/+4
| | | | | | | You can omit @number suffix when specifying /export option, but you can do that only for stdcall functions. llvm-svn: 207809
* Fix FreeBSD buildbots.Rui Ueyama2014-05-011-1/+1
| | | | llvm-svn: 207802
* [PECOFF] Do not call addDllExport to mutate export descriptors.Rui Ueyama2014-05-011-4/+4
| | | | | | This is more efficient than before. llvm-svn: 207791
* [ELF] Fix the file look up algorithm used in the linker script GROUP command.Simon Atanasyan2014-05-011-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | In general the linker scripts's GROUP command works like a pair of command line options --start-group/--end-group. But there is a difference in the files look up algorithm. The --start-group/--end-group commands use a trivial approach: a) If the path has '-l' prefix, add 'lib' prefix and '.a'/'.so' suffix and search the path through library search directories. b) Otherwise, use the path 'as-is'. The GROUP command implements more compicated approach: a) If the path has '-l' prefix, add 'lib' prefix and '.a'/'.so' suffix and search the path through library search directories. b) If the path does not have '-l' prefix, and sysroot is configured, and the path starts with the / character, and the script being processed is located inside the sysroot, search the path under the sysroot. Otherwise, try to open the path in the current directory. If it is not found, search through library search directories. https://www.sourceware.org/binutils/docs-2.24/ld/File-Commands.html The patch reviewed by Shankar Easwaran, Rui Ueyama. llvm-svn: 207769
* [PECOFF] Fix exported symbol name.Rui Ueyama2014-05-012-3/+13
| | | | | | | | When creating a .lib file, we should strip the leading underscore, but should not strip stdcall atsign suffix. Otherwise produced .lib files cannot be linked. llvm-svn: 207729
* [PECOFF] Add a test for lib.exe subcommand.Rui Ueyama2014-05-011-18/+43
| | | | | | | | | | | | Previously the input file for the lib.exe command would be removed as soon as the command exits, so we couldn't write a test to check the file contents are correct. This patch adds /lldmoduledeffile: option to retain a copy of the temporary file at the given file path, so that you can see the file if you want. llvm-svn: 207727
* [ELF] Return result from the function ASAP.Simon Atanasyan2014-04-301-16/+6
| | | | | | No functional changes. llvm-svn: 207690
* [ELF] Factor out the code builds a library search directory pathSimon Atanasyan2014-04-301-14/+12
| | | | | | | | into the separate function. No functional changes. llvm-svn: 207689
* [PECOFF] Fix priority of locally imported symbols.Rui Ueyama2014-04-301-1/+2
| | | | | | | | | | | | | | | | Linker should create _imp_ symbols for local use only when such symbols cannot be resolved in any other way. If it overrides real imported symbols, such symbols remain virtually unresolved without error, causing odd issues. I observed that a program linked with LLD entered an infinite loop before reaching main() because of this issue. This patch moves the virtual file creating _imp_ symbols to the very end of the input file list. Previously, the file is at the end of the library file group. Linker might revisit the group many times, so it was not really at the end of the input file list. llvm-svn: 207605
* [Mips] Re-order include headers.Simon Atanasyan2014-04-291-3/+1
| | | | llvm-svn: 207495
* [Mips] Implement emitting of R_MIPS_REL32 relocations:Simon Atanasyan2014-04-297-25/+269
| | | | | | | | | | | | | 1. Re-implement PLT entries and dynamic relocations emitting to keep PLT and relocations table in a consistent state. 2. Initialize st_value and st_other fields for dynamic symbols table entry if this entry corresponds to an external function which address is taken in a non-PIC executable. In that case the st_value field holds an address of the function's PLT entry. Also set STO_MIPS_PLT bit in the st_other field. llvm-svn: 207494
* [Mips] 80 column.Simon Atanasyan2014-04-291-2/+6
| | | | llvm-svn: 207493
* [PECOFF] /export accepts non-decorated symbols.Rui Ueyama2014-04-291-1/+8
| | | | | | | You usually have to specify the exact name of a symbol to export it, but for stdcall functions you can omit the @numbers suffix. llvm-svn: 207491
* [PECOFF] Fix _imp_ implicit symbols.Rui Ueyama2014-04-292-20/+88
| | | | | | | | | | | | | | | | | | | | | | | Implicit symbol for local use implemented in r207141 was not fully compatible with MSVC link.exe. In r207141, I implemented the feature in such way that implicit symbols are defined only when they are exported with /EXPORT option. After that I found that implicit symbols are defined not only for dllexported symbols but for all defined symbols. Actually _imp_ implicit symbols have no relationship with the dllexport feature. You could add _imp_ to any symbol to get a pointer to the symbol, whether the symbol is dllexported or not. It looks pretty weird to me but that's what we want if link.exe behaves that way. Here is a bit about the implementation: Creating all implicit symbols beforehand is going to be a huge waste of resource. This feature is rarely used, and MSVC link.exe even prints out a warning message when it finds this feature is being used. So we create implicit symbols on demand. There is an archive file that creates implicit symbols when they are needed. llvm-svn: 207476
* [Mips] Convert MipsDynamicSymbolTable to the class template.Simon Atanasyan2014-04-283-10/+14
| | | | llvm-svn: 207435
* [Mips] Add the 'const' qualifier to the couple RelocationPass classSimon Atanasyan2014-04-281-4/+4
| | | | | | member functions. llvm-svn: 207434
* [PECOFF] Add /IMPLIB command line option.Rui Ueyama2014-04-252-7/+9
| | | | | | This option is to override the default import file path. llvm-svn: 207175
* [PECOFF] Assign unique ordinals to __imp_ symbols.Rui Ueyama2014-04-251-4/+10
| | | | llvm-svn: 207168
* [PECOFF] Do not copy-construct ExportDesc.Rui Ueyama2014-04-241-1/+1
| | | | llvm-svn: 207154
* [PECOFF] Define implicit symbols for exported ones.Rui Ueyama2014-04-241-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is to fix a compatibility issue with MSVC link.exe as to use of dllexported symbols inside DLL. A DLL exports two symbols for a function. One is non-decorated one, and the other is with __imp_ prefix. The former is a function that you can directly call, and the latter is a pointer to the function. These dllexported symbols are created by linker for programs that link against the DLL. So, I naturally believed that __imp_ symbols become available when you once create a DLL and link against it, but they don't exist until then. And that's not true. MSVC link.exe is smart enough to allow users to use __imp_ symbols locally. That is, if a symbol is specified with /export option, it implicitly creates a new symbol with __imp_ prefix as a pointer to the exported symbol. This feature allows the following program to be linked and run, although _imp__hello is not defined in this code. #include <stdio.h> __declspec(dllexport) void hello(void) { printf("Hello\n"); } extern void (*_imp__hello)(void); int main() { _imp__hello(); return 0; } MSVC link.exe prints out the following warning when linking it. LNK4217: locally defined symbol _hello imported in function _main Using __imp_ symbols locally is I think not a good coding style. One should just take an address using "&" operator rather than appending __imp_ prefix. However, there are programs in the wild that depends on this link.exe's behavior, so we need this feature. llvm-svn: 207141
* [PECOFF] Allow symbols not starting with '_' in x86Rui Ueyama2014-04-241-1/+2
| | | | | | | | Not all symbols are decorated with an underscore in x86. You can write undecorated symbols in assembly, for example. Thus this assertion is too strong. llvm-svn: 207125
* [PECOFF] Skip IMAGE_SYM_DEBUG sections correctly.Rui Ueyama2014-04-221-6/+8
| | | | | | | | | We don't use sections with IMAGE_SYM_DEBUG attribute so we basically want to the symbols for them when reading symbol table. When we skip them, we need to skip auxiliary symbols too. Otherwise weird error would happen because aux symbols would be interpreted as regular ones. llvm-svn: 206931
* [Modules] Fix potential ODR violations by sinking the DEBUG_TYPEChandler Carruth2014-04-223-6/+6
| | | | | | | | | | definition below all of the header #include lines, LLD edition. IF you want to know more details about this, you can see the recent commits to Debug.h in LLVM. This is just the LLD segment of a cleanup I'm doing globally for this macro. llvm-svn: 206851
* [ELF] Fix GNU_RELRO section name.Rui Ueyama2014-04-181-1/+1
| | | | llvm-svn: 206560
* [ELF] Support --defsym=<symbol>=<symbol>.Rui Ueyama2014-04-161-0/+75
| | | | | | | | | | | | | | | | | | | | | Currently LLD supports --defsym only in the form of --defsym=<symbol>=<integer>, where the integer is interpreted as the absolute address of the symbol. This patch extends it to allow other symbol name to be given as an RHS value. If a RHS value is a symbol name, the LHS symbol will be defined as an alias for the RHS symbol. Internally, a LHS symbol is represented as a zero-size defined atom who has an LayoutAfter reference to an undefined atom, whose name is the RHS value. Everything else is already implemented -- Resolver will resolve the undefined symbol, and the layout pass will layout the two atoms at the same location. Looks like it's working fine. Note that GNU LD supports --defsym=<symbol>=<symbol>+<addend>. That feature is out of scope of this patch. Differential Revision: http://reviews.llvm.org/D3332 llvm-svn: 206417
* [Mips] Emit PLT entries for more static relocations.Simon Atanasyan2014-04-161-6/+18
| | | | llvm-svn: 206358
OpenPOWER on IntegriCloud