Fixed in rL356847.

• add GOT.mem and GOT.func

• cleanup

• revert fast isel changes
• cleanup

• cleanup test
• revert fast isel changes for now

• add more tests

• remove_lld_todo

@dschuff i updated the PR description and hopefully this a good to go now?

This is needed for my PIC change: https://reviews.llvm.org/D54647

-rebase

• split out lambda

• clang-format
• remove 64

• feedback
• cleanup

• cleaup test code
• add MC test

• improve test

• selection dag working

• and more!

• remove even more

• isel start

revert parts

update

• fix error

Mostly lgtm

In D59281#1429109, @sunfish wrote:

In the future, atomics will likely be enabled by default, however it will continue to be desirable to be able to select non-shared memory, since non-shared memory doesn't require a maximum size, and since it can make interoperating with JS and other external code simpler. Is it possible to structure this patch in a way that supports this?

Add test

• clang-format
• comments
• cleanup

• comments

This change is now ready for review. Please note that this is only the most simple support for .so file. All it does is add each library to the "needed" section, it doesn't do any symbol resolution against the shared libraries yet.

• clang-format

• remove assert

• add test

• move to mono-repo

• move to monorepo

rebase

• move to mono-repo

• rebase

ping..

In D58956#1421823, @aardappel wrote:

It would be awesome if we could somehow force tabs to be errors, frankly. Or is there some other automatic way to detect them? Or show them in code review?

I know this changes is abandoned, but I am in favor of it.

• revert

In D59074#1421059, @sbc100 wrote:

In D59074#1421058, @pcc wrote:

Turns own that IsUsedInRegularObject is set for lazy (archive) symbols.

Should we fix that, then?

It correct according to the comment in ELF/Symbols.h:

// True if the symbol was used for linking and thus need to be added to the    
// output file's symbol table. This is true for all symbols except for         
// unreferenced DSO symbols and bitcode symbols that are unreferenced except   
// by other bitcode objects.                                                   
unsigned IsUsedInRegularObj : 1;

I guess objects in archives are still regular objects.

In D59074#1421058, @pcc wrote:

Turns own that IsUsedInRegularObject is set for lazy (archive) symbols.

Should we fix that, then?