Page MenuHomePhabricator

ruiu (Rui Ueyama)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 18 2013, 12:16 AM (363 w, 6 d)

Recent Activity

Yesterday

ruiu added a comment to D77672: Support a few more SPARCv9 relocations.

Improving SPARC support is fine as long as we don't need to do something special for the ISA. But since no active maintainers have a SPARC machine, it is hard to find bugs in SPARC. Can you add tests for the new relocations?

For what it's worth, I have access to various SPARC machines, so can test things as needed. Also, the GCC compile farm (https://cfarm.tetaneutral.net/machines/list/) has multiple SPARC machines; in particular, gcc202 is managed by us Debian sparc64 porters, and has a beefy 64 cores and 64 GiB on a T5 (which, unlike some of the earlier UltraSparcs, has decent single-threaded performance too), so it should be easy for maintainers to test things if they're interested enough. The compile farm is a great resource in general that is really easy to sign up for and most don't know about.

Tue, Apr 7, 10:50 PM · Restricted Project
ruiu added inline comments to D77659: [ELF][test] Add reproduce test for dependent libraries.
Tue, Apr 7, 9:46 PM · Restricted Project
ruiu added a comment to D77672: Support a few more SPARCv9 relocations.

Improving SPARC support is fine as long as we don't need to do something special for the ISA. But since no active maintainers have a SPARC machine, it is hard to find bugs in SPARC. Can you add tests for the new relocations?

Tue, Apr 7, 9:13 PM · Restricted Project

Mon, Apr 6

ruiu accepted D68065: Propeller: LLD Support for Basic Block Sections.

I had a VC meeting wtih maskray to discuss concerns he has, and he is ok with this patch. I'm fine with this patch too. I'll give a final LGTM to unblock you. Please make the following changes and submit. Thanks!

Mon, Apr 6, 8:44 PM · Restricted Project, lld

Sun, Apr 5

ruiu added inline comments to D77484: [Vector] Pass VectLib to LTO backend so TLI build correct vector function list.
Sun, Apr 5, 10:25 PM · Restricted Project

Wed, Apr 1

ruiu added inline comments to D76839: [lld-macho] Extend SyntheticSections to cover all segment load commands.
Wed, Apr 1, 8:42 PM · Restricted Project
ruiu added a comment to D76742: [lld] Add basic symbol table output.

We usually construct a section in three phases as follows:

Wed, Apr 1, 8:09 PM · Restricted Project
ruiu added inline comments to D76839: [lld-macho] Extend SyntheticSections to cover all segment load commands.
Wed, Apr 1, 2:43 AM · Restricted Project
ruiu added a comment to D77087: [lld-macho] Require executables to link against libSystem.dylib.

The tests for lld/mach-o definitely need to be able to run on non-macOS platforms. We generally run all tests on all kind of bots, so yes, you cannot assume that /usr/lib/libSystem.B.dylib exists. Actually you shouldn't make any assumptions on the existence or absence of any external file.

Wed, Apr 1, 1:36 AM · Restricted Project

Tue, Mar 31

ruiu accepted D76999: [ELF] Print symbols with non-default versions for better "undefined symbol" diagnostics.

LGTM

Tue, Mar 31, 11:05 PM · Restricted Project
ruiu added inline comments to D76999: [ELF] Print symbols with non-default versions for better "undefined symbol" diagnostics.
Tue, Mar 31, 10:00 PM · Restricted Project
ruiu added inline comments to D76252: [lld-macho] Add basic support for linking against dylibs.
Tue, Mar 31, 10:00 PM · Restricted Project
ruiu accepted D76908: [lld-macho] Add support for emitting dylibs with a single symbol.

LGTM

Tue, Mar 31, 10:00 PM · Restricted Project
ruiu added inline comments to D76839: [lld-macho] Extend SyntheticSections to cover all segment load commands.
Tue, Mar 31, 10:00 PM · Restricted Project
ruiu added inline comments to D76977: [lld-macho] Implement basic export trie.
Tue, Mar 31, 9:27 PM · Restricted Project
ruiu accepted D76960: [lld][WebAssembly] Early error if output file cannot be created..

LGTM

Tue, Mar 31, 9:27 PM · Restricted Project
ruiu added a comment to D77087: [lld-macho] Require executables to link against libSystem.dylib.

What do you mean by "upstream"?

Tue, Mar 31, 9:27 PM · Restricted Project
ruiu added a comment to D77087: [lld-macho] Require executables to link against libSystem.dylib.

Hm, I was just trying to emulate ld64's behavior, but I agree that this one might not be very useful. Regardless of whether we throw an error, though, having special-case code to emit it at test time seems useful, if only so we can run the resulting executables on macOS.

Tue, Mar 31, 8:23 PM · Restricted Project
ruiu added inline comments to D76977: [lld-macho] Implement basic export trie.
Tue, Mar 31, 8:23 PM · Restricted Project
ruiu added a comment to D77086: [lld-macho][easy] Use substitution for test.

Manual line wrapping is kind of a chore, and since we don't have an autoformatter for these files, we'd end up with lines that are too short after doing that sed operation. Plus I was also considering adding -Z to the base %lld substitution to ensure that we never write tests which accidentally depend on system libraries.

Tue, Mar 31, 8:22 PM · Restricted Project
ruiu added inline comments to D77187: [lld][WebAssembly] Add initial support for -Map/--print-map.
Tue, Mar 31, 8:22 PM · Restricted Project
ruiu added inline comments to D76977: [lld-macho] Implement basic export trie.
Tue, Mar 31, 2:10 AM · Restricted Project
ruiu accepted D77006: [lld-macho] Support reading of universal binaries.

LGTM

Tue, Mar 31, 2:10 AM · Restricted Project
ruiu added a comment to D77087: [lld-macho] Require executables to link against libSystem.dylib.

I'm not sure why we need this. Could you elaborate a bit? If all executables need to be linked against /usr/lib/libSystem.B.dylib and if a user didn't pass -lSystem, isn't it still their fault and not ours, no?

Tue, Mar 31, 2:10 AM · Restricted Project
ruiu added inline comments to D76908: [lld-macho] Add support for emitting dylibs with a single symbol.
Tue, Mar 31, 1:37 AM · Restricted Project
ruiu added a comment to D76839: [lld-macho] Extend SyntheticSections to cover all segment load commands.

Overall I'd like to see more comments. All functions and classes that are not trivial should have a comment explaining what they represent or what they are supposed to do.

Tue, Mar 31, 1:04 AM · Restricted Project

Mon, Mar 30

ruiu added inline comments to D76742: [lld] Add basic symbol table output.
Mon, Mar 30, 11:58 PM · Restricted Project
ruiu added inline comments to D76252: [lld-macho] Add basic support for linking against dylibs.
Mon, Mar 30, 11:58 PM · Restricted Project
ruiu added a comment to D76885: [lld][COFF][ELF][WebAssembly] Replace --[no-]threads /threads[:no] with --threads={1,2,...} /threads:{1,2,...}.

Instead of --threads=all, can you use --threads? --threads has several advantages over --threads=all:

Mon, Mar 30, 11:25 PM · Restricted Project
ruiu added a comment to D77086: [lld-macho][easy] Use substitution for test.

I'm actually not in favor of this change as I want to keep test files self-contained. I'd stick with lld -flavor darwinnew. Once the new lld/mach-o is ready, you can run sed with s/lld -flavor darwinnew/lld.ld64/g to replace all occurrences of lld -flavor darwinnew. So I don't see a need for abstraction here.

Mon, Mar 30, 10:22 PM · Restricted Project
ruiu added a comment to D75382: [lld] Initial commit for new Mach-O backend.

Can you submit this change? I think this is good enough as an initial commit, and you can make further changes on top of this.

Mon, Mar 30, 10:22 PM · Restricted Project
ruiu added a comment to D76885: [lld][COFF][ELF][WebAssembly] Replace --[no-]threads /threads[:no] with --threads={1,2,...} /threads:{1,2,...}.

Do we need all? It's the default behavior, so if you want to specify --threads=all, you can just delete it altogether.

Mon, Mar 30, 9:18 PM · Restricted Project

Tue, Mar 24

ruiu added a comment to D76668: [lld][ELF] Ignore __start/__stop symbols in section groups during GC.

lld's --gc-sections behavior didn't take start/stop symbols into consideration, but it turned out some programs didn't work with that algorithm, so we (I believe rafael or pcc) added the logic to preserve sections for start/stop symbols. Therefore, I think we can't ignore start/stop symbols at the section gc stage and need to at least keep the existing logic.

Tue, Mar 24, 10:56 PM · Restricted Project

Mon, Mar 23

ruiu added a comment to D76668: [lld][ELF] Ignore __start/__stop symbols in section groups during GC.

Although I understand your concrete example, I'm not sure if this is logically correct. This patch is written based on the following assumption:

Mon, Mar 23, 11:57 PM · Restricted Project

Wed, Mar 18

ruiu added a comment to D76252: [lld-macho] Add basic support for linking against dylibs.

As to the tests, we generally don't use yaml2obj to write tests (except for COFF which is historically using yaml2obj). If you are using yaml2obj just until lld itself is able to produce files used by tests, I don't think you have to spend too much time writing tests in yaml2obj. Instead, you can submit binary files for now and then replace them later.

Hi @ruiu,

I'm just doing a quick drive by comment here, since I'm not really doing much LLD work at the moment myself. Why are you opposed to yaml2obj being used to generate test inputs instead of llvm-mc? Surely yaml2obj gives more direct control over what is actually in the input? In llvm-mc you only have relatively limited control over some things like relocations.

It seems to me like yaml2obj is a better instrument to be using for the vast majority of test cases, with only a limited set using llvm-mc as a catch-all test to make sure that we don't miss some odd interaction. Of course, this is all making the assumption that mach-o yaml2obj is straightforward to use (I have a feeling it might not be).

You can write tests with yaml2obj too, but if you take a look at ELF, don't you think that we can write fine-grained test cases with llvm-mc too? I want tests to be realistic input files, and in that respect, test cases written in llvm-mc are better than ones in obj2yaml.

I also found that tests written in llvm-mc are easier to maintain. For example, imagine that you already have a test file containing tests for relocations and you want to add a new type of relocation. Then, you'll have to not only add a new relocation to a yaml file but also add a few extra bytes with some addend at a right bit-wise position to the .text section, which isn't easy to do by hand. So, I guess you'll end up writing a test in assembly, compile it and then convert it to yaml -- but if that's the case, we should write tests in assembly in the first place. On the other hand, I didn't find a test case that is easy to write in yaml but not in assembly. There might be some, but that shouldn't be too many. So it looks to me that llvm-mc is a better choice to write test files.

I think there is no universal approach and I agree with both of you. For example, for testing relocation relaxations I'd definetely prefer using YAML because it is important to get the particular op-codes and relocations in the inputs.
It is probaly easier to use YAMLs rather than using llvm-mc and verifying inputs with objdump or alike. For many other things, using llvm-mc is probably a bit more natural.

What I do not agree with is that commiting binaries is OK. From my one last experience after working on
yaml2obj and converting LLVM binaries, I found them are source of all kind of problems. If we can use yaml2obj for that out of box, I think we should.
If we can't then probably it is OK to use binaries temporary, but probably a bug should be reported about lacking features of yaml2obj and the
test should have a FIXME comment with a link.

Wed, Mar 18, 3:46 AM · Restricted Project
ruiu accepted D75382: [lld] Initial commit for new Mach-O backend.

LGTM

Wed, Mar 18, 3:13 AM · Restricted Project
ruiu added a comment to D76252: [lld-macho] Add basic support for linking against dylibs.

As to the tests, we generally don't use yaml2obj to write tests (except for COFF which is historically using yaml2obj). If you are using yaml2obj just until lld itself is able to produce files used by tests, I don't think you have to spend too much time writing tests in yaml2obj. Instead, you can submit binary files for now and then replace them later.

Hi @ruiu,

I'm just doing a quick drive by comment here, since I'm not really doing much LLD work at the moment myself. Why are you opposed to yaml2obj being used to generate test inputs instead of llvm-mc? Surely yaml2obj gives more direct control over what is actually in the input? In llvm-mc you only have relatively limited control over some things like relocations.

It seems to me like yaml2obj is a better instrument to be using for the vast majority of test cases, with only a limited set using llvm-mc as a catch-all test to make sure that we don't miss some odd interaction. Of course, this is all making the assumption that mach-o yaml2obj is straightforward to use (I have a feeling it might not be).

Wed, Mar 18, 2:41 AM · Restricted Project
ruiu added inline comments to D75382: [lld] Initial commit for new Mach-O backend.
Wed, Mar 18, 1:04 AM · Restricted Project
ruiu added inline comments to D75382: [lld] Initial commit for new Mach-O backend.
Wed, Mar 18, 12:31 AM · Restricted Project

Tue, Mar 17

ruiu added a comment to D76252: [lld-macho] Add basic support for linking against dylibs.

The no-id-dylink.s test needs yaml2obj to generate an invalid dylib, but yeah I can replace the other two with binary blobs

We are trying to avoid binary blob in LLVM binary utilities + lld. @grimar has done lots of work in this area. Can you figure out how much effort it takes to implement relevant features in yaml2obj Mach-O? I hope we can avoid binary blobs early.

Tue, Mar 17, 6:56 PM · Restricted Project
ruiu added a comment to D76252: [lld-macho] Add basic support for linking against dylibs.

As to the tests, we generally don't use yaml2obj to write tests (except for COFF which is historically using yaml2obj). If you are using yaml2obj just until lld itself is able to produce files used by tests, I don't think you have to spend too much time writing tests in yaml2obj. Instead, you can submit binary files for now and then replace them later.

Tue, Mar 17, 12:52 AM · Restricted Project

Mon, Mar 16

ruiu added inline comments to D76252: [lld-macho] Add basic support for linking against dylibs.
Mon, Mar 16, 10:43 PM · Restricted Project
ruiu accepted D68065: Propeller: LLD Support for Basic Block Sections.

It looks like there's no activities in the llvm-dev thread and this review thread, so I'll reiterate my LGTM, as I think we should not block this change, in particular given that the corresponding change to LLVM has been submitted.

Mon, Mar 16, 3:09 AM · Restricted Project, lld
ruiu committed rGa2923b2a1e45: Implement CET Shadow Stack (Intel Controlflow Enforcement Technology) support… (authored by ruiu).
Implement CET Shadow Stack (Intel Controlflow Enforcement Technology) support…
Mon, Mar 16, 2:10 AM
ruiu closed D70606: LLD: CET shadow stack support on Windows.
Mon, Mar 16, 2:10 AM · Restricted Project, lld

Fri, Mar 13

ruiu added a comment to D70606: LLD: CET shadow stack support on Windows.

That's discussed on llvm-dev, and I think that even though we should ideally extend
the tool to handle the new bit, it shouldn't block this patch. This patch adds a missing
flag that was added to MSVC link.exe recently.

Fri, Mar 13, 4:24 AM · Restricted Project, lld
ruiu added inline comments to D70557: [lld][COFF] Add support for /map.
Fri, Mar 13, 12:50 AM · Restricted Project, lld
ruiu added a comment to D75382: [lld] Initial commit for new Mach-O backend.

Jez and Shoaib, if you guys have more patches on this one, you can send them now for review if you want.

Fri, Mar 13, 12:50 AM · Restricted Project

Thu, Mar 12

ruiu added a comment to D75382: [lld] Initial commit for new Mach-O backend.

LGTM

Thu, Mar 12, 11:19 PM · Restricted Project
ruiu accepted D76109: [ELF] Correct error message when OUTPUT_FORMAT is used.

LGTM

Thu, Mar 12, 10:36 PM · Restricted Project

Tue, Mar 10

ruiu accepted D75382: [lld] Initial commit for new Mach-O backend.

Thank you very much for doing this!

Tue, Mar 10, 3:46 AM · Restricted Project
ruiu accepted D75742: [Object] Support large archive members.

LGTM

Tue, Mar 10, 12:09 AM · Restricted Project

Mon, Mar 9

ruiu accepted D70557: [lld][COFF] Add support for /map.

LGTM

Mon, Mar 9, 11:21 PM · Restricted Project, lld

Mar 9 2020

ruiu added inline comments to D75382: [lld] Initial commit for new Mach-O backend.
Mar 9 2020, 2:07 AM · Restricted Project
ruiu accepted D75809: [lld] Add .clang-tidy to customize readability-identifier-naming.{Member,Parameter,Variable}Case => camelBack.

LGTM

Mar 9 2020, 2:07 AM · Restricted Project

Mar 5 2020

ruiu added a comment to D69665: [llvm-ar] Fix llvm-ar response file reading on Windows.

Yeah, that's another type of consistency, and if you feel that that consistency is more important, feel free to make a change to wasm-ld so that the tool defaults to runtime OS's rsp-quoting style.

Mar 5 2020, 11:30 PM · Restricted Project
ruiu accepted D74773: [LLD][Debuginfo] create DWARFContext only once for the same object file..

LGTM

Mar 5 2020, 11:30 PM · debug-info, Restricted Project
ruiu added a comment to D70557: [lld][COFF] Add support for /map.

Overall looking good.

Mar 5 2020, 10:58 PM · Restricted Project, lld
ruiu added a comment to D70606: LLD: CET shadow stack support on Windows.

I tried to apply your patch to submit, but it looks like your test file (llvm/test/tools/llvm-readobj/COFF/Inputs/has_cet.exe) cannot be downloaded from Phabricator. Can you upload a file somewhere and send me the URL? Or you can encode with base64 and send it to me via email.

Mar 5 2020, 9:49 PM · Restricted Project, lld

Mar 4 2020

ruiu added a comment to D75648: [lld] Rename -flavor darwin to -flavor old-darwin.

But we are going to call the new one as mach_o2 until mach_o can be removed and rename the new one mach_o once it's done, no? If so, I don't think we need to rename only this part at this moment. We can rename them all at once later.

Mar 4 2020, 6:31 PM · Restricted Project
ruiu added inline comments to D75382: [lld] Initial commit for new Mach-O backend.
Mar 4 2020, 1:46 AM · Restricted Project
ruiu added a comment to D69665: [llvm-ar] Fix llvm-ar response file reading on Windows.

lld's default choices are different among platforms as we are trying to match the native linkers' behaviors. So is true to clang; if you invoke clang as clang-cl (which is a Microsoft-compatible compiler driver), the value is set to Windows by default. I think we didn't have any other policy than this, as we are creating drop-in replacements.

Mar 4 2020, 1:13 AM · Restricted Project

Mar 3 2020

ruiu added inline comments to D74773: [LLD][Debuginfo] create DWARFContext only once for the same object file..
Mar 3 2020, 11:01 PM · debug-info, Restricted Project
ruiu accepted D70606: LLD: CET shadow stack support on Windows.

LGTM with this change.

Mar 3 2020, 10:21 PM · Restricted Project, lld
ruiu accepted D75577: [lld][WebAssembly] Add support for --rsp-quoting.

LGTM

Mar 3 2020, 10:21 PM · Restricted Project

Mar 1 2020

ruiu added a comment to D75382: [lld] Initial commit for new Mach-O backend.

This seems a pretty straightforward port of the existing backends. Very nice!

Mar 1 2020, 9:08 PM · Restricted Project

Feb 27 2020

ruiu added a comment to D68065: Propeller: LLD Support for Basic Block Sections.

If Sriraman has already resolved all Fangrui's comments, I can reiterate my LGTM, as the new code is well written, and I don't have a specific concern. I actually think that this is an interesting feature and in some degree a natural extension of -ffunction-sections.

Feb 27 2020, 8:32 PM · Restricted Project, lld

Feb 24 2020

ruiu accepted D75100: [ELF] Support archive:file syntax in input section descriptions.

LGTM

Feb 24 2020, 10:53 PM · Restricted Project
ruiu added inline comments to D74773: [LLD][Debuginfo] create DWARFContext only once for the same object file..
Feb 24 2020, 6:52 PM · debug-info, Restricted Project
ruiu accepted D68065: Propeller: LLD Support for Basic Block Sections.

LGTM

Feb 24 2020, 6:33 PM · Restricted Project, lld

Feb 13 2020

ruiu added a comment to D68065: Propeller: LLD Support for Basic Block Sections.

Overall looking good, but it looks like this test file doesn't cover all relocations you want to handle. You are handling the following relocations, and compared to that the test file seems too small.

Feb 13 2020, 11:38 PM · Restricted Project, lld

Feb 12 2020

ruiu added inline comments to D68065: Propeller: LLD Support for Basic Block Sections.
Feb 12 2020, 11:47 PM · Restricted Project, lld

Feb 11 2020

ruiu added inline comments to D68065: Propeller: LLD Support for Basic Block Sections.
Feb 11 2020, 11:12 PM · Restricted Project, lld

Feb 7 2020

ruiu added a comment to D73497: lld: Propeller framework part I.

As we discussed today, this patch didn't make sense to me at first sight, as it redoes a lot of things that we already implemented to lld (e.g. making a new "config" object only for this thing, defining a new type of Symbol (SymbolEntry) even though we have Symbol class, etc.) But the thing is that this code is intended to be used by some external tool. If that's the case, you should consider moving this to LLVM instead and use the code from lld and your tool.

Feb 7 2020, 4:25 AM · Restricted Project
ruiu added a comment to D68065: Propeller: LLD Support for Basic Block Sections.

Here is a summary of today's discussion:

Feb 7 2020, 4:25 AM · Restricted Project, lld

Jan 16 2020

ruiu added a comment to D72803: Make lld cmake not compute commit revision twice.

LGTM

Jan 16 2020, 6:33 PM · Restricted Project

Jan 9 2020

ruiu added a comment to D59780: Support Intel Control-flow Enforcement Technology.

@MaskRay Please rebase and upload it with me and other people as reviewers, then I'll LGTM.

Jan 9 2020, 6:46 AM · Restricted Project

Jan 6 2020

ruiu added inline comments to D72183: [ELF][PPC64] Add --lax-call-lacks-nop.
Jan 6 2020, 12:53 AM · Restricted Project
ruiu added inline comments to D72183: [ELF][PPC64] Add --lax-call-lacks-nop.
Jan 6 2020, 12:35 AM · Restricted Project

Jan 5 2020

ruiu added inline comments to D72183: [ELF][PPC64] Add --lax-call-lacks-nop.
Jan 5 2020, 10:43 PM · Restricted Project
ruiu accepted D72196: [lld] Fix trivial typos in comments.

LGTM

Jan 5 2020, 10:34 PM · Restricted Project, lld
ruiu added inline comments to D71981: [LLD] [COFF] Don't error out on duplicate absolute symbols with the same value.
Jan 5 2020, 10:25 PM · Restricted Project

Dec 19 2019

ruiu accepted D71735: [ELF] Don't suggest an alternative spelling for a symbol in a discarded section.

LGTM

Dec 19 2019, 9:46 PM · Restricted Project

Dec 18 2019

ruiu accepted D71679: [LLD] [COFF] Fix reporting duplicate errors for absolute symbols.

LGTM

Dec 18 2019, 8:02 PM · Restricted Project

Dec 17 2019

ruiu accepted D71631: [ELF] writePlt: replace parameters gotPltEntryAddr and index with `const Symbol &`. NFC.

I think I like sym over s as a function parameter name whose scope is not very narrow, but except that LGTM.

Dec 17 2019, 11:49 PM · Restricted Project
ruiu accepted D71101: [lld][RISCV] Use an e_flags of 0 if there are only binary input files..
Dec 17 2019, 6:36 PM · Restricted Project
ruiu added inline comments to D71632: [lld][WebAssembly] Fail if bitcode objects are pulled in after LTO.
Dec 17 2019, 6:27 PM · Restricted Project

Dec 16 2019

ruiu added inline comments to D69665: [llvm-ar] Fix llvm-ar response file reading on Windows.
Dec 16 2019, 10:44 PM · Restricted Project
ruiu added a comment to D71437: [PDB] Print the most redundant type record indices with /summary.

Yeah, I guess that showing that tip line would help users.

Dec 16 2019, 9:04 PM · Restricted Project
ruiu accepted D71060: [LLD][ELF] Add time-trace to ELF LLD (2/2).

LGTM

Dec 16 2019, 8:54 PM · Restricted Project
ruiu accepted D71548: Fix time trace multi threaded support with LLVM_ENABLE_THREADS=OFF.

LGTM

Dec 16 2019, 8:18 PM · Restricted Project
ruiu added a comment to D71509: [ELF][PPC64] Implement IPLT code sequence for non-preemptible IFUNC.

Looks good but I want someone from PPC side to take a look at this change.

Dec 16 2019, 8:18 PM · Restricted Project

Dec 15 2019

ruiu accepted D71518: [ELF] Delete relOff from TargetInfo::writePLT.

LGTM

Dec 15 2019, 10:18 PM · Restricted Project
ruiu added inline comments to D71520: [ELF] Rename .plt to .iplt and decrease EM_PPC{,64} alignment of .glink to 4.
Dec 15 2019, 9:42 PM · Restricted Project
ruiu accepted D71519: [ELF] Add IpltSection.

LGTM

Dec 15 2019, 9:42 PM · Restricted Project

Dec 13 2019

ruiu added a comment to D71161: [ELF] Update st_size when merging a common symbol with a shared symbol.

This behavior seems to make sense but is very subtle. Can you add a comment to explain what this is doing?

Dec 13 2019, 12:35 AM · Restricted Project

Dec 12 2019

ruiu committed rG69da7e29dea6: Revert an accidental commit af5ca40b47b3e85c3add81ccdc0b787c4bc355ae (authored by ruiu).
Revert an accidental commit af5ca40b47b3e85c3add81ccdc0b787c4bc355ae
Dec 12 2019, 10:19 PM
ruiu committed rG6faf8bdcc469: Update the man page (authored by ruiu).
Update the man page
Dec 12 2019, 10:19 PM
ruiu committed rGaf5ca40b47b3: temporary (authored by ruiu).
temporary
Dec 12 2019, 10:19 PM
ruiu closed D71385: Update the man page.
Dec 12 2019, 10:19 PM · Restricted Project