Page MenuHomePhabricator

sidbav (Sidharth Baveja)
User

Projects

User does not belong to any projects.

User Details

User Since
May 20 2020, 7:11 AM (6 w, 4 d)

Recent Activity

Thu, Jul 2

sidbav updated the summary of D82927: Intergerate Loop Peeling into Loop Fusion.
Thu, Jul 2, 8:40 AM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address @Meinersbur comments. Remove the Loop Peeling files since it is unrelated to fusion. The Loop Peeling changes revision is here: D83056

Thu, Jul 2, 8:37 AM · Restricted Project
sidbav created D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Thu, Jul 2, 8:37 AM · Restricted Project

Tue, Jun 30

sidbav added a comment to D80580: Separate Peeling Properties into its own struct.

@fhahn added the followup patch D82927. Also ping at reviewers of this patch! Thanks!

Tue, Jun 30, 7:00 PM · Restricted Project
sidbav created D82927: Intergerate Loop Peeling into Loop Fusion.
Tue, Jun 30, 7:00 PM · Restricted Project
sidbav updated the diff for D80580: Separate Peeling Properties into its own struct.

Some NFC modifications.

Tue, Jun 30, 6:59 PM · Restricted Project

May 27 2020

sidbav updated the diff for D80583: Create utility function to Merge Adjacent Basic Blocks.

Update function description comments based on review.

May 27 2020, 2:09 PM · Restricted Project
sidbav added a comment to D80583: Create utility function to Merge Adjacent Basic Blocks.

Updated comments based on review.

May 27 2020, 2:09 PM · Restricted Project
sidbav added inline comments to D80583: Create utility function to Merge Adjacent Basic Blocks.
May 27 2020, 7:33 AM · Restricted Project
sidbav updated the diff for D80583: Create utility function to Merge Adjacent Basic Blocks.

Updated the code based on review.

May 27 2020, 7:33 AM · Restricted Project

May 26 2020

sidbav added a comment to D80580: Separate Peeling Properties into its own struct.

@fhahn I think a follow-up patch will take a little bit of time to develop so here is a quick example.

for (i = 0; i < 10; ++i)
  a[i] = a[i] + 3;
for (j = 1; j < 10; ++j)
  b[j] = b[j] + 5;
May 26 2020, 2:44 PM · Restricted Project
sidbav added a comment to D80580: Separate Peeling Properties into its own struct.

Could you elaborate why this change is desired? Is there a problem with getting the unrolling preferences and checking the peeling properties in that way? If that's the case it would be good to provide some context (e.g. in form of a follow-on patch that uses the new PeelingPreferences).

May 26 2020, 1:39 PM · Restricted Project
sidbav updated the summary of D80580: Separate Peeling Properties into its own struct.
May 26 2020, 1:38 PM · Restricted Project
sidbav added a reviewer for D80583: Create utility function to Merge Adjacent Basic Blocks: etiotto.
May 26 2020, 1:05 PM · Restricted Project
sidbav created D80583: Create utility function to Merge Adjacent Basic Blocks.
May 26 2020, 1:05 PM · Restricted Project
sidbav updated the summary of D80580: Separate Peeling Properties into its own struct.
May 26 2020, 1:05 PM · Restricted Project
sidbav created D80580: Separate Peeling Properties into its own struct.
May 26 2020, 1:05 PM · Restricted Project