Page MenuHomePhabricator

sidbav (Sidharth Baveja)
User

Projects

User does not belong to any projects.

User Details

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

Recent Activity

Thu, Jan 14

sidbav requested review of D94717: [LoopNest] Consider loop nest with inner loop guard using outer loop induction variable to be perfect .
Thu, Jan 14, 1:37 PM · Restricted Project

Wed, Jan 13

sidbav requested review of D94619: [SplitEdge] Update to handle cases which were not previously considered when the edge is critical..
Wed, Jan 13, 11:02 AM · Restricted Project

Thu, Jan 7

sidbav committed rG048f184ee488: [SplitEdge] Add new parameter to SplitEdge to name the newly created basic block (authored by sidbav).
[SplitEdge] Add new parameter to SplitEdge to name the newly created basic block
Thu, Jan 7, 6:51 AM
sidbav closed D94176: [SplitEdge] Add new parameter to SplitEdge to name the newly created basic block.
Thu, Jan 7, 6:51 AM · Restricted Project

Wed, Jan 6

sidbav updated the diff for D94176: [SplitEdge] Add new parameter to SplitEdge to name the newly created basic block.

Made a small error in the previous diff, updated the minor typo.

Wed, Jan 6, 8:58 AM · Restricted Project
sidbav updated the diff for D94176: [SplitEdge] Add new parameter to SplitEdge to name the newly created basic block.

Pass in the BBName Parameter to the critical edge case as well.

Wed, Jan 6, 8:39 AM · Restricted Project
sidbav requested review of D94176: [SplitEdge] Add new parameter to SplitEdge to name the newly created basic block.
Wed, Jan 6, 7:01 AM · Restricted Project

Aug 1 2020

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.

Aug 1 2020, 6:08 AM · Restricted Project

Jul 31 2020

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…
Jul 31 2020, 11:32 AM
sidbav closed D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Jul 31 2020, 11:32 AM · Restricted Project

Jul 30 2020

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

Ping!

Jul 30 2020, 8:15 AM · Restricted Project

Jul 23 2020

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…
Jul 23 2020, 2:04 PM
sidbav closed D82927: Intergerate Loop Peeling into Loop Fusion.
Jul 23 2020, 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.

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

Address review comments

Jul 23 2020, 1:11 PM · Restricted Project
sidbav added inline comments to D83056: Separate the Loop Peeling Utilities from the Loop Unrolling Utilities .
Jul 23 2020, 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

Jul 23 2020, 8:28 AM · Restricted Project
sidbav added inline comments to D82927: Intergerate Loop Peeling into Loop Fusion.
Jul 23 2020, 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.

Jul 23 2020, 5:48 AM · Restricted Project

Jul 21 2020

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…
Jul 21 2020, 9:01 AM
sidbav closed D82927: Intergerate Loop Peeling into Loop Fusion.
Jul 21 2020, 9:01 AM · Restricted Project

Jul 20 2020

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 .
Jul 20 2020, 7:08 AM · Restricted Project

Jul 17 2020

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

Address Florian 's comments.

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

Address Bardia's comments

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

Ping @ reviewers! Thanks!

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

Ping @ reviewers! Thanks!

Jul 17 2020, 6:34 AM · Restricted Project

Jul 16 2020

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

Address review comments

Jul 16 2020, 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