emaste (Ed Maste)
User

Projects

User Details

User Since
Jul 24 2013, 5:36 AM (269 w, 6 d)

Recent Activity

Yesterday

emaste added a comment to D50294: [Driver] Use -gdwarf-3 by default for FreeBSD.

I'm using this change: https://github.com/emaste/freebsd/commit/1c3deab6d518feb1a7e88de5b342a139e4022a21

Tue, Sep 25, 11:10 AM
emaste accepted D52507: Fix FreeBSD build after r342633..

The FreeBSD build bot was down for a little over a week due to an IP address issue, but is back now: http://lab.llvm.org:8011/builders/lldb-amd64-ninja-freebsd11

Tue, Sep 25, 10:19 AM

Mon, Sep 24

emaste updated subscribers of D52430: Introduce a flag to warn when IFUNC symbols are used with text relocations..
Mon, Sep 24, 12:59 PM
emaste committed rLLD342887: Correct RISC-V link in release notes.
Correct RISC-V link in release notes
Mon, Sep 24, 7:52 AM
emaste committed rL342887: Correct RISC-V link in release notes.
Correct RISC-V link in release notes
Mon, Sep 24, 7:52 AM

Mon, Sep 17

emaste added a comment to D50297: Align AArch64 and i386 image base to superpage.

In theory this change shouldn't affect the amount of memory used by an application

Mon, Sep 17, 12:19 PM

Fri, Sep 14

emaste abandoned D52096: lld: add -z nodump support.

Agreed - when I submitted it I thought it might be uncommon but worth supporting, but the evidence Mark found suggests it is a single-purpose feature that is not relevant to the platforms lld supports.

Fri, Sep 14, 10:31 AM
emaste added a comment to D52096: lld: add -z nodump support.

Thanks for finding that Mark - based on that IMO this feature is not worth supporting.

Fri, Sep 14, 8:50 AM
emaste created D52096: lld: add -z nodump support.
Fri, Sep 14, 7:56 AM
emaste committed rLLD342239: lld: add -z interpose support.
lld: add -z interpose support
Fri, Sep 14, 7:29 AM
emaste committed rL342239: lld: add -z interpose support.
lld: add -z interpose support
Fri, Sep 14, 7:29 AM
emaste closed D52094: lld: add -z interpose support.
Fri, Sep 14, 7:29 AM
emaste updated the diff for D52094: lld: add -z interpose support.

keep flags in order

Fri, Sep 14, 7:24 AM
emaste updated the diff for D52094: lld: add -z interpose support.

Reuse existing FLAGS: test

Fri, Sep 14, 7:22 AM
emaste added inline comments to D52094: lld: add -z interpose support.
Fri, Sep 14, 7:22 AM
emaste updated subscribers of D52094: lld: add -z interpose support.

See also https://reviews.freebsd.org/D17172 for a lld 6 version intended for commit to FreeBSD's in-tree lld.

Fri, Sep 14, 7:06 AM
emaste created D52094: lld: add -z interpose support.
Fri, Sep 14, 7:04 AM

Aug 5 2018

emaste added a comment to D39322: [lld] Support RISC-V.

@ruiu what do you suggest as the next step here?

Aug 5 2018, 12:03 PM · lld

Jul 30 2018

emaste added a comment to D49992: [ELF][ARM] Add Arm ABI names for float ABI ELF Header flags.

LGTM. I made the same change in FreeBSD r336745.

Jul 30 2018, 9:07 AM
emaste added a comment to D49938: [ELF] Add missing options to ld.lld.1.

FWIW I don't have a strong opinion on | vs newline to separate aliases, but if we are going to go with the newline -compact and .Pp approach I'd suggest making that change as a separate commit first so that the actual content changes here are easier to see/review.

Jul 30 2018, 6:34 AM
emaste added inline comments to D49938: [ELF] Add missing options to ld.lld.1.
Jul 30 2018, 6:31 AM
emaste added a comment to D49938: [ELF] Add missing options to ld.lld.1.

For reference I had a discussion on markup for option aliases with Ingo Schwarze and he wrote:

Jul 30 2018, 6:24 AM

Jul 4 2018

emaste added a comment to D41821: [MIPS] Disassemble the 0xefefefef trap padding used by LLD.

I'm going to implement both parts of the plan.

Jul 4 2018, 11:08 AM

May 29 2018

emaste added a comment to D31528: [ELF][MIPS] Multi-GOT implementation.

Ping?

@ruiu could I use a post-commit review option in that case?

May 29 2018, 10:55 AM · lld
emaste removed a reviewer for D31528: [ELF][MIPS] Multi-GOT implementation: espindola.
May 29 2018, 10:25 AM · lld

May 25 2018

emaste added inline comments to D47073: Document and Enforce new Host Compiler Policy.
May 25 2018, 5:58 AM

May 10 2018

emaste accepted D46690: [ELF] - Improve the test cases for notes sections..

After the second look, I think it is ok to add check here that we still keep non-a section and keep its SHT_NOTE flag for safety though. I'll update the patch.

May 10 2018, 6:36 AM
emaste accepted D46690: [ELF] - Improve the test cases for notes sections..

note-noalloc2.s is close to my original test, I think that's reasonable.

May 10 2018, 5:48 AM
emaste committed rLLD331973: [ELF] Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
[ELF] Omit PT_NOTE for SHT_NOTE without SHF_ALLOC
May 10 2018, 4:16 AM
emaste committed rL331973: [ELF] Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
[ELF] Omit PT_NOTE for SHT_NOTE without SHF_ALLOC
May 10 2018, 4:16 AM
emaste closed D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
May 10 2018, 4:16 AM

May 9 2018

emaste updated the diff for D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
May 9 2018, 8:22 PM
emaste added a comment to D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
  1. what is the content of .debug-ghc-link-info? If the section contains an optional, human-readable string, they might have just used .comment instead.
May 9 2018, 11:16 AM
emaste added a comment to D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.

Committed to FreeBSD in r333401

May 9 2018, 4:19 AM
emaste added a comment to D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.

Yeah, @espindola was added by a Herald rule.

May 9 2018, 4:03 AM

May 8 2018

emaste added a comment to D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.

@arichardson @dim I'm also looking for approval to commit the change to FreeBSD (perhaps in advance of committing upstream), with a bump of the FreeBSD-specific version to LLD_REVISION_STRING "326565-1200002" - I might as well solicit that via this review.

May 8 2018, 7:53 PM
emaste created D46623: Omit PT_NOTE for SHT_NOTE without SHF_ALLOC.
May 8 2018, 7:41 PM

Apr 28 2018

emaste committed rLLD331133: Add -warn-backrefs (r329636) to lld's man page.
Add -warn-backrefs (r329636) to lld's man page
Apr 28 2018, 7:23 PM
emaste committed rL331133: Add -warn-backrefs (r329636) to lld's man page.
Add -warn-backrefs (r329636) to lld's man page
Apr 28 2018, 7:22 PM
emaste added inline comments to D45195: Add --check-library-dependency to maintain compatibility with other linkers.
Apr 28 2018, 7:16 PM

Apr 25 2018

emaste added a comment to D46080: Pack symbols a bit more.

Nice :)

Apr 25 2018, 1:58 PM
emaste added a comment to D45652: Asan, fix FreeBSD support.

Do we know why pthread_{get,set}specific fails now? And would it make sense to use tls more broadly (not just on FreeBSD)?

Apr 25 2018, 9:25 AM
emaste added a reviewer for D45652: Asan, fix FreeBSD support: dim.
Apr 25 2018, 9:09 AM

Apr 21 2018

emaste committed rL330504: Temporarily skip Go TestExpressions on FreeBSD as it hangs.
Temporarily skip Go TestExpressions on FreeBSD as it hangs
Apr 21 2018, 7:02 AM
emaste committed rL330500: FreeBSD: propagate error to user if memory access fails.
FreeBSD: propagate error to user if memory access fails
Apr 21 2018, 4:27 AM

Apr 9 2018

emaste added a comment to D31528: [ELF][MIPS] Multi-GOT implementation.

Happy belated birthday Multi-GOT patch

Apr 9 2018, 1:21 PM · lld

Mar 15 2018

emaste added a comment to D40623: Correctly set reserved bits for UUID version 4..

LGTM

Mar 15 2018, 8:57 AM

Mar 9 2018

emaste added a comment to D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD.

Thanks for the review. As for the -Wl,-z,origin part, I'd rather leave that alone unless we have a clear indication that either it is not necessary for any of the supported FreeBSD and DragonFly BSD versions, or that it can be passed unconditionally without breaking any of the supported systems (and which linkers accept it).

Mar 9 2018, 2:25 PM

Mar 8 2018

emaste accepted D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD.

LGTM for FreeBSD

Mar 8 2018, 6:20 AM
emaste committed rL327000: Use ellipsis ... to indicate omitted commands.
Use ellipsis ... to indicate omitted commands
Mar 8 2018, 5:54 AM

Mar 7 2018

emaste added a comment to D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD.

ld(1) might accept -z origin, but why do we want it for other OSes? I don't see it used in the NetBSD dynamic ELF loader. DF_ORIGIN is just defined but not used by anything.

Mar 7 2018, 8:18 PM
emaste added a comment to D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD.

Uconditionally for all Unices? I don't think so.

Mar 7 2018, 4:12 PM

Mar 5 2018

emaste added a comment to D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD.

LGTM from the FreeBSD perspective

Mar 5 2018, 8:23 AM
emaste added a reviewer for D43751: [cmake] Append -Wl,-rpath-link conditionally to GNULD: dim.
Mar 5 2018, 8:16 AM

Mar 1 2018

emaste committed rLLD326536: Update man page for long opts that accept = after r326506.
Update man page for long opts that accept = after r326506
Mar 1 2018, 6:59 PM
emaste committed rL326536: Update man page for long opts that accept = after r326506.
Update man page for long opts that accept = after r326506
Mar 1 2018, 6:59 PM
emaste committed rL326463: Correct man page description for --section-start.
Correct man page description for --section-start
Mar 1 2018, 11:05 AM
emaste committed rLLD326463: Correct man page description for --section-start.
Correct man page description for --section-start
Mar 1 2018, 11:05 AM
emaste committed rLLD326459: Where possible use --long-opt=value in lld man page.
Where possible use --long-opt=value in lld man page
Mar 1 2018, 10:42 AM
emaste committed rL326459: Where possible use --long-opt=value in lld man page.
Where possible use --long-opt=value in lld man page
Mar 1 2018, 10:42 AM
emaste committed rLLD326446: Correct markup and nits in ld.lld man page.
Correct markup and nits in ld.lld man page
Mar 1 2018, 8:04 AM
emaste committed rL326446: Correct markup and nits in ld.lld man page.
Correct markup and nits in ld.lld man page
Mar 1 2018, 8:04 AM

Feb 27 2018

emaste added a comment to D43643: [RFC] Sceptre a Spectre variant 1 detector.

Is the pass intentionally named "Sceptre"? That's amusing, but kind of confusing.

Feb 27 2018, 10:28 AM

Feb 26 2018

emaste committed rL326166: Mark test_*int*_t_dwarf as failing on FreeBSD.
Mark test_*int*_t_dwarf as failing on FreeBSD
Feb 26 2018, 6:56 PM
emaste committed rL326139: Revert r326134 due to broken buildbot.
Revert r326134 due to broken buildbot
Feb 26 2018, 2:41 PM
emaste committed rL326134: Mark test_*int*_t_dwarf as failing on FreeBSD.
Mark test_*int*_t_dwarf as failing on FreeBSD
Feb 26 2018, 2:16 PM
emaste added inline comments to D32167: Add support for type units (.debug_types) to LLDB in a way that is compatible with DWARF 5.
Feb 26 2018, 10:36 AM

Feb 22 2018

emaste committed rL325856: remove FreeBSD xfail from lit TestCallStdStringFunction.
remove FreeBSD xfail from lit TestCallStdStringFunction
Feb 22 2018, 6:53 PM

Feb 21 2018

emaste added a comment to D42516: [llvm-objcopy] Add support for large indexes.
Feb 21 2018, 7:14 PM
emaste added a comment to D42516: [llvm-objcopy] Add support for large indexes.

One other point - it would be good to compare the performance of llvm-objcopy to GNU objcopy in this case, and if we're a lot worse, run a profiler at some point to identify what we're doing that makes things worse.

Feb 21 2018, 3:52 PM

Feb 19 2018

emaste added a comment to D43168: Don't use -ldl on BSD.

In FreeBSD libpthread.so is a symlink to libthr; -pthread and -lpthread should both work.

Feb 19 2018, 9:23 PM

Feb 16 2018

emaste added a comment to D43370: Add initial XRay support for NetBSD.

This LGTM

Feb 16 2018, 6:46 AM · Restricted Project

Feb 9 2018

emaste committed rL324739: Add IMPLEMENTATION NOTES describing lld's .a handling in the man page.
Add IMPLEMENTATION NOTES describing lld's .a handling in the man page
Feb 9 2018, 7:38 AM
emaste committed rLLD324739: Add IMPLEMENTATION NOTES describing lld's .a handling in the man page.
Add IMPLEMENTATION NOTES describing lld's .a handling in the man page
Feb 9 2018, 7:38 AM
emaste closed D43072: ld.lld.1: add IMPLEMENTATION NOTES section documenting lld's handling.
Feb 9 2018, 7:38 AM

Feb 8 2018

emaste updated subscribers of D43072: ld.lld.1: add IMPLEMENTATION NOTES section documenting lld's handling.
Feb 8 2018, 5:47 PM
emaste created D43072: ld.lld.1: add IMPLEMENTATION NOTES section documenting lld's handling.
Feb 8 2018, 7:16 AM

Feb 7 2018

emaste committed rL324528: Document --build-id=fast in ld.lld.1.
Document --build-id=fast in ld.lld.1
Feb 7 2018, 1:24 PM
emaste committed rLLD324528: Document --build-id=fast in ld.lld.1.
Document --build-id=fast in ld.lld.1
Feb 7 2018, 1:24 PM
emaste closed D43039: ld.lld.1: document --build-id=fast.
Feb 7 2018, 1:24 PM
emaste created D43039: ld.lld.1: document --build-id=fast.
Feb 7 2018, 1:05 PM
emaste added inline comments to D42963: Add initial ld.lld.1 man page.
Feb 7 2018, 12:45 PM
emaste committed rL324512: Add ld.lld.1 man page, describing lld's ELF linker.
Add ld.lld.1 man page, describing lld's ELF linker
Feb 7 2018, 12:40 PM
emaste committed rLLD324512: Add ld.lld.1 man page, describing lld's ELF linker.
Add ld.lld.1 man page, describing lld's ELF linker
Feb 7 2018, 12:40 PM
emaste closed D42963: Add initial ld.lld.1 man page.
Feb 7 2018, 12:40 PM
emaste added a comment to D42963: Add initial ld.lld.1 man page.

Thanks, I'll commit this later today. Not sure how I ended up with 01 instead of O1 but have fixed those locally.

Feb 7 2018, 12:08 PM
emaste added inline comments to D42963: Add initial ld.lld.1 man page.
Feb 7 2018, 6:21 AM
emaste added inline comments to D42963: Add initial ld.lld.1 man page.
Feb 7 2018, 6:16 AM
emaste added a comment to D42963: Add initial ld.lld.1 man page.

PDF version of current version:

Feb 7 2018, 6:16 AM
emaste updated the diff for D42963: Add initial ld.lld.1 man page.
Feb 7 2018, 6:16 AM

Feb 6 2018

emaste added inline comments to D42963: Add initial ld.lld.1 man page.
Feb 6 2018, 6:04 PM
emaste added a comment to D42963: Add initial ld.lld.1 man page.

PDF rendered version of the man page:

Feb 6 2018, 6:04 PM
emaste added inline comments to D42963: Add initial ld.lld.1 man page.
Feb 6 2018, 6:01 PM
emaste updated the diff for D42963: Add initial ld.lld.1 man page.

Incorporate review feedback

Feb 6 2018, 5:59 PM
emaste updated the diff for D42963: Add initial ld.lld.1 man page.
  • Update copyright and license statement
  • Remove internal -full-shutdown option
Feb 6 2018, 9:40 AM
emaste created D42963: Add initial ld.lld.1 man page.
Feb 6 2018, 6:51 AM

Jan 26 2018

emaste added inline comments to D42550: Put the header in the first PT_LOAD even if that PT_LOAD has a LMAExpr.
Jan 26 2018, 6:35 AM

Jan 25 2018

emaste added a comment to D31528: [ELF][MIPS] Multi-GOT implementation.

We have a chance to celebrate the first anniversary of this patch in a couple of months. Is it possible to escape this dubious holiday?

Jan 25 2018, 2:10 PM · lld
emaste added inline comments to D42550: Put the header in the first PT_LOAD even if that PT_LOAD has a LMAExpr.
Jan 25 2018, 11:20 AM

Jan 11 2018

emaste accepted D41635: Align SHT_NOBITS sections is they are the first on a PT_LOAD.

Seems fine to me

Jan 11 2018, 7:37 PM