Page MenuHomePhabricator

carwil (Carey Williams)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 5 2018, 8:33 AM (85 w, 11 h)

Recent Activity

Thu, Aug 1

carwil accepted D65516: [NFC] Test commit, corrected some spelling in comment.

NFC test commit. LGTM.

Thu, Aug 1, 5:04 AM · Restricted Project

Thu, Jul 25

carwil added a comment to D65000: [ARM] Set default alignment to 64bits.

test case missing A8 aside this looks ok to me. Would like to see if there are any comments from the Pacific time zone.

Thu, Jul 25, 5:44 AM · Restricted Project, Restricted Project

Mar 22 2019

carwil committed rG2c3c9ca4ba26: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent (authored by carwil).
[ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent
Mar 22 2019, 9:20 AM
carwil committed rL356776: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.
[ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent
Mar 22 2019, 9:20 AM
carwil committed rC356776: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.
[ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent
Mar 22 2019, 9:20 AM
carwil closed D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.
Mar 22 2019, 9:19 AM · Restricted Project

Mar 15 2019

carwil updated the diff for D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.

Done. I've added an extra parameter over what you might expect as classifyArgumentTypes doesn't seem to consider AAPCS16, whereas classifyReturnTypes does.
I've also renamed the variable to make the distinction between it and the function clearer.

Mar 15 2019, 3:31 AM · Restricted Project

Mar 14 2019

carwil updated the diff for D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.

I got bit a confused earlier. That does actually seem like correct behaviour (once we're no longer able to treat the struct as a homogeneous aggregate).
I've tightened up the conditional a bit as it wouldn't have previously accounted for the case where you had mfloat-abi=hard with an AAPCS (non VFP) attribute. Although in this instance, the backend does seem to be correcting/ignoring that, it's still best we don't blindly rely on such behaviour.

Mar 14 2019, 9:48 AM · Restricted Project

Mar 8 2019

carwil added a comment to D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.

Not sure how to write a testcase off the top of my head... have you tried homogeneous aggregates with more than two elements?

Mar 8 2019, 2:58 AM · Restricted Project

Mar 7 2019

carwil added reviewers for D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent: srhines, efriedma, peter.smith, psmith.
Mar 7 2019, 7:19 AM · Restricted Project
carwil created D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent.
Mar 7 2019, 7:17 AM · Restricted Project

Feb 21 2019

carwil added a comment to D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.

Not at all, thank you your comments.

Feb 21 2019, 8:33 AM

Feb 15 2019

carwil updated subscribers of D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.

Hi @eli.friedman, are you still happy to review this?

Feb 15 2019, 2:23 AM

Feb 5 2019

carwil updated the diff for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.

Removed the complicated frame pointer/ffixed combination errors in favour of an always on warning (in a new group, so it can be silenced).

Feb 5 2019, 4:00 AM
carwil updated the diff for D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.

Fixed an oversight where R6 could be used as the base pointer, even with ffixed-r6 specified.

Feb 5 2019, 3:57 AM

Jan 24 2019

carwil accepted D57088: Add information about unit tests to the testing guide.

Thanks for the changes! LGTM. Approved.

Jan 24 2019, 6:52 AM

Jan 23 2019

carwil updated the diff for D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.

Added more test coverage, notably that to catch issues with using reserved registers for alignment and making up Thumb reg deficit.

Jan 23 2019, 9:26 AM
carwil updated the diff for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.

More tests, and better handling of argument combination errors.

Jan 23 2019, 9:23 AM
carwil added a comment to D57088: Add information about unit tests to the testing guide.

Thanks for posting this! Always good to doc updates.
Bit of _very_ opinionated bit of bike-shedding. Content otherwise LGTM!

Jan 23 2019, 3:33 AM

Jan 18 2019

carwil added a comment to D56305: [AArch64] Support reserving arbitrary general purpose registers.

Hi, we're currently working on similar functionality for the ARM backend, so I got pointed to take a look at this patch.
Looks good to me, I quite like the tablegen improvements, but agree with trong's comments.

Jan 18 2019, 10:03 AM · Restricted Project, Restricted Project

Jan 15 2019

carwil updated the diff for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.

Added cfe-commits.

Jan 15 2019, 3:26 AM
carwil updated the diff for D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.

Added llvm-commits.

Jan 15 2019, 2:35 AM

Jan 14 2019

carwil updated the diff for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.

Added some regression tests for invalid -ffixed-reg combinations and the declaration of (un)supported variables.

Jan 14 2019, 10:20 AM

Jan 11 2019

carwil updated the diff for D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.
  • Re-implementation using subtarget features over metadata.
  • Reduction of register range to R5-R11.
  • Additional checks to fix issues with spilling registers for stack alignment.
Jan 11 2019, 12:45 PM
carwil updated the diff for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.
  • Re-implementation using subtarget features over metadata.
  • Reduction of register range to R5-R11.
  • Additional checks to fix issues with spilling registers for stack alignment.
  • Added clang errors for conflicting options with reserved registers.
Jan 11 2019, 12:40 PM

Dec 21 2018

carwil added a reviewer for D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM: amilendra_arm.
Dec 21 2018, 7:48 AM
carwil added a reviewer for D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM: amilendra_arm.
Dec 21 2018, 7:48 AM
carwil created D56005: [RFC] [LLVM] Allocatable Global Register Variables for ARM.
Dec 21 2018, 7:46 AM
carwil created D56003: [RFC] [CFE] Allocatable Global Register Variables for ARM.
Dec 21 2018, 7:41 AM

Dec 17 2018

carwil committed rC349335: [Docs] Expand -fstack-protector and -fstack-protector-all.
[Docs] Expand -fstack-protector and -fstack-protector-all
Dec 17 2018, 2:15 AM
carwil committed rL349335: [Docs] Expand -fstack-protector and -fstack-protector-all.
[Docs] Expand -fstack-protector and -fstack-protector-all
Dec 17 2018, 2:14 AM
carwil closed D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info.
Dec 17 2018, 2:14 AM · Restricted Project

Dec 12 2018

carwil updated the diff for D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info.

Reworded -fstack-protector-all to bring it in line with the changes to the other two options.

Dec 12 2018, 2:10 AM · Restricted Project

Dec 11 2018

carwil updated the diff for D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info.

Make thopre's suggested changes.

Dec 11 2018, 12:43 AM · Restricted Project

Dec 7 2018

carwil added reviewers for D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info: thopre, petpav01, jmagee.
Dec 7 2018, 9:36 AM · Restricted Project
carwil updated the diff for D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info.

Edited Options.td directly, rather than the generated docs file.

Dec 7 2018, 9:31 AM · Restricted Project
carwil created D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info.
Dec 7 2018, 5:16 AM · Restricted Project

Jan 22 2018

carwil committed rL323118: [AArch64] optimise v4f16 fcmps to utilise vector instructions.
[AArch64] optimise v4f16 fcmps to utilise vector instructions
Jan 22 2018, 6:19 AM
carwil closed D41772: [AArch64] optimise v4f16 FCMPs to utilise vector instructions.
Jan 22 2018, 6:19 AM
carwil updated the diff for D41772: [AArch64] optimise v4f16 FCMPs to utilise vector instructions.

Missing colon

Jan 22 2018, 6:18 AM
carwil updated the diff for D41772: [AArch64] optimise v4f16 FCMPs to utilise vector instructions.
Jan 22 2018, 5:40 AM

Jan 19 2018

carwil committed rL322958: Test commit.
Test commit
Jan 19 2018, 8:57 AM

Jan 16 2018

carwil updated the diff for D41772: [AArch64] optimise v4f16 FCMPs to utilise vector instructions.

Improved code formatting based on review

Jan 16 2018, 10:50 AM

Jan 5 2018

carwil created D41772: [AArch64] optimise v4f16 FCMPs to utilise vector instructions.
Jan 5 2018, 9:22 AM