Page MenuHomePhabricator

Joe (Joseph Faulls)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 1 2019, 1:37 AM (51 w, 1 d)

Recent Activity

Apr 8 2020

Joe added a comment to D76354: [WIP][RISCV][GlobalISel] Legalize types for ALU operations.

LGTM, but probably wait for someone more experienced to upvote :)

Apr 8 2020, 6:27 AM · Restricted Project

Mar 20 2020

Joe added inline comments to D74977: [WIP][RISCV][GlobalISel] Add lowerFormalArguments for calling convention.
Mar 20 2020, 8:05 AM · Restricted Project
Joe added a comment to D76354: [WIP][RISCV][GlobalISel] Legalize types for ALU operations.

What about G_OR and G_XOR? Is there a reason why they're not listed?

Mar 20 2020, 7:32 AM · Restricted Project

Mar 17 2020

Joe added a comment to D75023: [WIP][RISCV][GlobalISel] Add lowerCall for calling convention.

If you're supporting a pointer value return type, should there be a test for this?

Mar 17 2020, 5:06 AM · Restricted Project

Dec 2 2019

Joe updated the diff for D68328: Fix occurrences that size and range of pointers are assumed to be the same..

Rebased on current head.

Dec 2 2019, 3:09 AM · Restricted Project, Restricted Project

Nov 25 2019

Joe added a comment to D68328: Fix occurrences that size and range of pointers are assumed to be the same..

LGTM,
Thanks

Nov 25 2019, 1:45 AM · Restricted Project, Restricted Project

Nov 14 2019

Joe updated the diff for D68328: Fix occurrences that size and range of pointers are assumed to be the same..

Addressed @igorb 's comments.

Nov 14 2019, 10:06 AM · Restricted Project, Restricted Project

Nov 13 2019

Joe updated the diff for D68328: Fix occurrences that size and range of pointers are assumed to be the same..

Updated diff:

Nov 13 2019, 6:26 AM · Restricted Project, Restricted Project

Nov 7 2019

Joe added a comment to D68328: Fix occurrences that size and range of pointers are assumed to be the same..

After further review and testing, I have found that changing the effective SCEV type to be the pointer index type does not work. Though neither does it work when assumed to be pointer size. In many places, these are assumed to be the same. I'm unsure of which way to go with this. Do I force effective SCEV type to be the pointer size and enforce this, or change it to be index size and enforce this?

Nov 7 2019, 8:09 AM · Restricted Project, Restricted Project

Nov 4 2019

Joe added a comment to D68328: Fix occurrences that size and range of pointers are assumed to be the same..

Ping

Nov 4 2019, 1:44 AM · Restricted Project, Restricted Project

Oct 4 2019

Joe added inline comments to D68328: Fix occurrences that size and range of pointers are assumed to be the same..
Oct 4 2019, 7:26 AM · Restricted Project, Restricted Project

Oct 2 2019

Joe added inline comments to D68328: Fix occurrences that size and range of pointers are assumed to be the same..
Oct 2 2019, 6:49 AM · Restricted Project, Restricted Project
Joe created D68328: Fix occurrences that size and range of pointers are assumed to be the same..
Oct 2 2019, 3:35 AM · Restricted Project, Restricted Project
Joe retitled D68328: Fix occurrences that size and range of pointers are assumed to be the same. from Fix occurances that size and range of pointers are assumed to be the same. to Fix occurrences that size and range of pointers are assumed to be the same..
Oct 2 2019, 3:35 AM · Restricted Project, Restricted Project