Page MenuHomePhabricator

sidbav (Sidharth Baveja)
User

Projects

User does not belong to any projects.

User Details

User Since
May 20 2020, 7:11 AM (12 w, 2 d)

Recent Activity

Sat, Aug 1

sidbav added a comment to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Hey @sckmodifier, please take a look at some comments after that one you just quoted. I added the NFC tag, but then in later discussion, it was determined it is best to remove the NFC tag since this change is in somewhat in a grey area.

Sat, Aug 1, 6:08 AM · Restricted Project

Fri, Jul 31

sidbav committed rGb7cfa6ca9283: [Loop Peeling] Separate the Loop Peeling Utilities from the Loop Unrolling… (authored by sidbav).
[Loop Peeling] Separate the Loop Peeling Utilities from the Loop Unrolling…
Fri, Jul 31, 11:32 AM
sidbav closed D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Fri, Jul 31, 11:32 AM · Restricted Project

Thu, Jul 30

sidbav added a comment to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Ping!

Thu, Jul 30, 8:15 AM · Restricted Project

Thu, Jul 23

sidbav committed rG38a82179315c: [Loop Fusion] Integrate Loop Peeling into Loop Fusion (re-land after fixing… (authored by sidbav).
[Loop Fusion] Integrate Loop Peeling into Loop Fusion (re-land after fixing…
Thu, Jul 23, 2:04 PM
sidbav closed D82927: Intergerate Loop Peeling into Loop Fusion.
Thu, Jul 23, 2:04 PM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Missed one comment from Bardia, address that comment.

Thu, Jul 23, 1:14 PM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address review comments

Thu, Jul 23, 1:11 PM · Restricted Project
sidbav added inline comments to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Thu, Jul 23, 8:29 AM · Restricted Project
sidbav updated the diff for D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Include all of the includes from before, and update patch based on changes in D82927

Thu, Jul 23, 8:28 AM · Restricted Project
sidbav added inline comments to D82927: Intergerate Loop Peeling into Loop Fusion.
Thu, Jul 23, 5:52 AM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address the ASAN build failures (major change is around line 795 in LoopFuse.cpp file). Also address some issues @MaskRay pointed out in lit tests.

Thu, Jul 23, 5:48 AM · Restricted Project

Tue, Jul 21

sidbav committed rGbb8850d34d60: [Loop Fusion] Integrate Loop Peeling into Loop Fusion Summary: This patch adds… (authored by sidbav).
[Loop Fusion] Integrate Loop Peeling into Loop Fusion Summary: This patch adds…
Tue, Jul 21, 9:01 AM
sidbav closed D82927: Intergerate Loop Peeling into Loop Fusion.
Tue, Jul 21, 9:01 AM · Restricted Project

Mon, Jul 20

sidbav retitled D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities from [NFC] Separate the Loop Peeling Utilities from the Loop Unrolling Utilities to Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Mon, Jul 20, 7:08 AM · Restricted Project

Fri, Jul 17

sidbav updated the diff for D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Address Florian 's comments.

Fri, Jul 17, 11:34 AM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address Bardia's comments

Fri, Jul 17, 10:17 AM · Restricted Project
sidbav added a comment to D82927: Intergerate Loop Peeling into Loop Fusion.

Ping @ reviewers! Thanks!

Fri, Jul 17, 6:34 AM · Restricted Project
sidbav added a comment to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Ping @ reviewers! Thanks!

Fri, Jul 17, 6:34 AM · Restricted Project

Thu, Jul 16

sidbav updated the diff for D83869: [Loop Simplify] Resolve an issue where metadata is not applied to a loop latch..

Address review comments

Thu, Jul 16, 8:19 AM · Restricted Project

Jul 15 2020

sidbav updated the diff for D83869: [Loop Simplify] Resolve an issue where metadata is not applied to a loop latch..

Address Bardia's comments

Jul 15 2020, 8:48 AM · Restricted Project
Herald added a project to D83869: [Loop Simplify] Resolve an issue where metadata is not applied to a loop latch.: Restricted Project.
Jul 15 2020, 6:51 AM · Restricted Project

Jul 10 2020

sidbav updated the diff for D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .

Minor update after making some modifications to D80580

Jul 10 2020, 2:17 PM · Restricted Project
sidbav updated the diff for D80580: [NFC] Separate Peeling Properties into its own struct.

Applying the patch in its previous state resulted in build failures. Updated patch to resolve all build failures. Major change is in the file llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp in the gatherPeelingPreferences function.

Jul 10 2020, 7:12 AM · Restricted Project

Jul 8 2020

sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address Bardia's comments.

Jul 8 2020, 7:08 AM · Restricted Project

Jul 7 2020

sidbav added inline comments to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Jul 7 2020, 2:49 PM · Restricted Project
sidbav updated the diff for D82927: Intergerate Loop Peeling into Loop Fusion.

Address Bardia's comments, major change is removing the loop-fusion-peel option.

Jul 7 2020, 2:43 PM · Restricted Project
sidbav added inline comments to D82927: Intergerate Loop Peeling into Loop Fusion.
Jul 7 2020, 2:00 PM · Restricted Project
sidbav retitled D80580: [NFC] Separate Peeling Properties into its own struct from Separate Peeling Properties into its own struct to [NFC] Separate Peeling Properties into its own struct.
Jul 7 2020, 10:44 AM · Restricted Project
sidbav retitled D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities from Separate the Loop Peeling Utilities from the Loop Unrolling Utilities to [NFC] Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Jul 7 2020, 10:44 AM · Restricted Project

Jul 2 2020

sidbav updated the summary of D82927: Intergerate Loop Peeling into Loop Fusion.
Jul 2 2020, 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

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

Jun 30 2020

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

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

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

Some NFC modifications.

Jun 30 2020, 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: [NFC] 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: [NFC] 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: [NFC] 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: [NFC] Separate Peeling Properties into its own struct.
May 26 2020, 1:05 PM · Restricted Project
sidbav created D80580: [NFC] Separate Peeling Properties into its own struct.
May 26 2020, 1:05 PM · Restricted Project