Page MenuHomePhabricator

brooks (Brooks Davis)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 20 2015, 9:29 AM (251 w, 9 h)

Recent Activity

Wed, Nov 6

Herald updated subscribers of D66725: [DAGCombiner][TargetLowering] Target hook for FCOPYSIGN arg cast folding.
Wed, Nov 6, 11:35 AM · Restricted Project

Jul 22 2019

brooks added a comment to D7104: On FreeBSD MIPS claim and pass down the -G# argument if present when linking..

@brooks, @arichardson do we still want to pursue this?

Jul 22 2019, 2:25 PM

Sep 25 2018

brooks created D52507: Fix FreeBSD build after r342633..
Sep 25 2018, 10:17 AM
brooks added a comment to D52507: Fix FreeBSD build after r342633..

See also: D51859

Sep 25 2018, 10:17 AM
brooks added a comment to D51859: [NFC] Turn "load dependent files" boolean into an enum .

Also broke the FreeBSD build:

Sep 25 2018, 10:05 AM

May 7 2018

brooks accepted D24867: Request init/fini array on FreeBSD 12 and later.

I do think we should make this change, but don't have a strong opinion on the timing. If we do it for 12 and not 13 then we should probably try to get it into a 6.0.2.

May 7 2018, 1:16 PM

May 6 2018

brooks updated the diff for D46503: [cmake] work around a race in Sphinx..
  • Correct whitespace.
May 6 2018, 9:42 AM
brooks added a comment to D46503: [cmake] work around a race in Sphinx..

I've received multiple reports of this issue from FreeBSD ports users, but have never encountered it myself. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227460

May 6 2018, 9:40 AM
brooks created D46503: [cmake] work around a race in Sphinx..
May 6 2018, 9:37 AM
brooks added a reviewer for D24867: Request init/fini array on FreeBSD 12 and later: arichardson.

Adding Alex as he made some related changes to CHERI Clang recently https://github.com/CTSRD-CHERI/clang/commit/3a648766deabb4ff7f95862213c3c99e7223363c.

May 6 2018, 9:11 AM

Jul 21 2015

brooks updated subscribers of D6231: [mips] Expand JAL instructions when PIC is enabled..
Jul 21 2015, 2:15 PM
brooks updated subscribers of D6267: [mips] Add assembler support for the .cprestore directive..
Jul 21 2015, 2:15 PM
brooks updated subscribers of D5626: [mips] Make the MipsAsmParser capable of knowing whether PIC mode is enabled or not..
Jul 21 2015, 2:15 PM
brooks updated subscribers of D9524: [mips] [IAS] Add support for the DLA pseudo-instruction and fix problems with DLI.
Jul 21 2015, 2:07 PM

May 29 2015

brooks added a comment to D10137: On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the assembler..

I may just not understand how render() works, but I tried it and it didn't seem to work to replace

May 29 2015, 4:23 PM
brooks updated the diff for D10137: On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the assembler..

Remove a spurious -mcpu from the test case.

May 29 2015, 4:21 PM
brooks added a comment to D9114: For FreeBSD on mips, pass -G options to the linker.
In D9114#181178, @dim wrote:

Even if this patch is incomplete with regards to the assembler, it is both necessary and sufficient to build and link programs with the current FreeBSD build system. It follows gcc's long established behavior as a linker and is harmless to other consumers so why not commit it?

Sure, but maybe -G for the linker is not completely useful, if the assembler also doesn't use it?

May 29 2015, 2:10 PM
brooks retitled D10137: On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the assembler. from to On {mips,mipsel,mips64,mips64el}-freebsd, we need to pass any -G option to the assembler..
May 29 2015, 2:10 PM
brooks added a comment to D9114: For FreeBSD on mips, pass -G options to the linker.

Even if this patch is incomplete with regards to the assembler, it is both necessary and sufficient to build and link programs with the current FreeBSD build system. It follows gcc's long established behavior as a linker and is harmless to other consumers so why not commit it?

May 29 2015, 12:27 PM

Jan 21 2015

brooks added a comment to D7104: On FreeBSD MIPS claim and pass down the -G# argument if present when linking..

Add cfe-commits per message

Jan 21 2015, 11:59 AM
brooks updated the diff for D7104: On FreeBSD MIPS claim and pass down the -G# argument if present when linking..

Add a test for not passing -G0 when it isn't an argument.

Jan 21 2015, 11:58 AM
brooks retitled D7104: On FreeBSD MIPS claim and pass down the -G# argument if present when linking. from to On FreeBSD MIPS claim and pass down the -G# argument if present when linking..
Jan 21 2015, 11:37 AM