Page MenuHomePhabricator

pgode (pankaj gode)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 12 2016, 7:26 AM (182 w, 19 h)

Recent Activity

Tue, Nov 26

pgode closed D70233: [WIP][Attributor] AAReachability Attribute.
Tue, Nov 26, 1:38 AM · Restricted Project

Fri, Nov 22

pgode added an edge to rG04945c92ce00: [WIP][Attributor] AAReachability Attribute: D70233: [WIP][Attributor] AAReachability Attribute.
Fri, Nov 22, 9:12 AM
pgode added 1 commit(s) for D70233: [WIP][Attributor] AAReachability Attribute: rG04945c92ce00: [WIP][Attributor] AAReachability Attribute.
Fri, Nov 22, 9:12 AM · Restricted Project
pgode set the repository for D70233: [WIP][Attributor] AAReachability Attribute to rG LLVM Github Monorepo.
Fri, Nov 22, 8:53 AM · Restricted Project

Wed, Nov 20

pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

Updated the patch as per review comments.

Wed, Nov 20, 10:55 PM · Restricted Project
pgode added inline comments to D70233: [WIP][Attributor] AAReachability Attribute.
Wed, Nov 20, 10:45 PM · Restricted Project
pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

Thanks for highlighting the nit. Corrected it.

Wed, Nov 20, 9:26 AM · Restricted Project
pgode added a comment to D70233: [WIP][Attributor] AAReachability Attribute.

It seems a bit weird for me to use IRPosition as a program point. Therefore, I think it is better to use Instruction instead of IRPosition. So the interface will be

bool isAssumedReachable(const Instruction &From, const Instruction &To)

What do you think?

Wed, Nov 20, 9:19 AM · Restricted Project

Tue, Nov 19

pgode retitled D70233: [WIP][Attributor] AAReachability Attribute from [WIP][NOT FOR COMMIT][Attributor] AAReachability Attribute to [WIP][Attributor] AAReachability Attribute.
Tue, Nov 19, 9:09 PM · Restricted Project

Sun, Nov 17

pgode added a comment to D70233: [WIP][Attributor] AAReachability Attribute.

Btw., do you have commit access?

Sun, Nov 17, 6:26 AM · Restricted Project
pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

Updated the function comments.

Sun, Nov 17, 3:26 AM · Restricted Project
pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

Thanks @jdoerfert for the helpful corrections. I have removed the files. (I should have removed them in the last patch.)
I will start working on the next part.

Sun, Nov 17, 2:59 AM · Restricted Project

Fri, Nov 15

pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

@jdoerfert Thank you suggesting the corrections.
Updating the corrected patch.

Fri, Nov 15, 10:41 AM · Restricted Project
pgode updated the diff for D70233: [WIP][Attributor] AAReachability Attribute.

Thank you @jdoerfert , @sstefan1 and @uenoku for your helpful comments.
Submitting updated patch with corrections.

Fri, Nov 15, 8:54 AM · Restricted Project

Thu, Nov 14

pgode retitled D70233: [WIP][Attributor] AAReachability Attribute from [WIP][NOT FOR COMMIT] AAReachability Attribute to [WIP][NOT FOR COMMIT][Attributor] AAReachability Attribute.
Thu, Nov 14, 4:10 AM · Restricted Project
pgode created D70233: [WIP][Attributor] AAReachability Attribute.
Thu, Nov 14, 4:10 AM · Restricted Project

Oct 3 2016

pgode accepted D25142: AArch64: Complete macroop fusion lists..

Thanks for splitting the code. LGTM.
Thanks.

Oct 3 2016, 10:23 PM

Oct 2 2016

pgode added a comment to D25142: AArch64: Complete macroop fusion lists..

Thanks for asking.
The first part of changes i.e. 'Fuse CMN, CMP, TST followed by Bcc', is applicable for Vulcan as well.
But, the second part i.e. 'Fuse ALU operations followed by CBZ/CBNZ' is not applicable for Vulcan, i.e. we don't fuse these. Is it possible to make this only Apple specific ?

Oct 2 2016, 10:53 AM

Jul 19 2016

pgode committed rL275978: [AArch64] PredictableSelectIsExpensive for Vulcan..
[AArch64] PredictableSelectIsExpensive for Vulcan.
Jul 19 2016, 7:38 AM
pgode closed D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan by committing rL275978: [AArch64] PredictableSelectIsExpensive for Vulcan..
Jul 19 2016, 7:37 AM
pgode updated the diff for D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan.

Removed the CacheLineSize (set to 64) for Vulcan, as three other parameters are required for LoopDataPrefetch to be effective.
Will include CacheLineSize & 3 other parameters in another patch, after they are finalized.

Jul 19 2016, 6:46 AM
pgode added inline comments to D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan.
Jul 19 2016, 6:44 AM

Jul 17 2016

pgode updated subscribers of D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan.
Jul 17 2016, 6:17 AM
pgode updated subscribers of D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan.
Jul 17 2016, 6:15 AM
pgode added reviewers for D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan: t.p.northover, rengolin, MatzeB.
Jul 17 2016, 6:15 AM
pgode retitled D22448: [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan from to [AArch64] Cache line size and PredictableSelectIsExpensive for Vulcan.
Jul 17 2016, 6:14 AM

Jul 8 2016

pgode committed rL274837: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.
[AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan
Jul 8 2016, 4:21 AM
pgode closed D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan by committing rL274837: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.
Jul 8 2016, 4:21 AM
pgode updated the diff for D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.

Updated diff.
Removed fusion of additional instructions for Vulcan.
Default instructions supported by 'Macrofusion subtarget feature' will be fused.

Jul 8 2016, 3:34 AM
pgode added a comment to D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.

The approach of adding a new sub-feature for Macro-op fusion, by categorizing the instructions (my presumption) doesn't seem a good option. It will end up adding too many subfeature such as FeatureMacroOpFusionArith/FeatureMacroOpFusionLogical. Please correct me.

Jul 8 2016, 2:14 AM
pgode added inline comments to D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.
Jul 8 2016, 12:27 AM

Jul 7 2016

pgode added inline comments to D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.
Jul 7 2016, 3:36 AM

Jul 6 2016

pgode retitled D22042: [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan from to [AArch64] Macro fusion of simple ALU ops with branches for Broadcom's Vulcan.
Jul 6 2016, 6:31 AM

Jun 29 2016

pgode committed rL274213: [AArch64] Add Broadcom Vulcan scheduling model..
[AArch64] Add Broadcom Vulcan scheduling model.
Jun 29 2016, 11:49 PM
pgode closed D21728: [AARCH64] Add Broadcom Vulcan scheduling model by committing rL274213: [AArch64] Add Broadcom Vulcan scheduling model..
Jun 29 2016, 11:49 PM
pgode added a comment to D21728: [AARCH64] Add Broadcom Vulcan scheduling model.

Corrected & updated the diff.

Jun 29 2016, 4:53 AM
pgode updated the diff for D21728: [AARCH64] Add Broadcom Vulcan scheduling model.

But the code looks good to me with the extra comment removed.

Removed the extra comments (comments in 3.11 & 3.12 FP Load/store).
Modified comments part of 3.6 & 3.7 (Load/Store) to say that they are valid for 3.11 & 3.12 (FP Load/Store) as well.

Jun 29 2016, 4:51 AM
pgode committed rL274114: [Driver][AArch64] Add support for Broadcom Vulcan core..
[Driver][AArch64] Add support for Broadcom Vulcan core.
Jun 29 2016, 3:07 AM
pgode closed D21501: [Driver] Add support for Broadcom Vulcan core by committing rL274114: [Driver][AArch64] Add support for Broadcom Vulcan core..
Jun 29 2016, 3:07 AM
pgode added a comment to D21728: [AARCH64] Add Broadcom Vulcan scheduling model.

Out of curiosity, what are the performance improvements with this model?

Actually, this is a functional patch and we are currently validating it on the simulators. So, we will update performance numbers later.

Jun 29 2016, 2:52 AM
pgode added a comment to D21728: [AARCH64] Add Broadcom Vulcan scheduling model.

Eric,
While submitting, I had added 'llvm-commits' as subscriber, as per the guidelines. (>> pgode added subscribers: echristo, aemerson, MatzeB, llvm-commits.)
Additionally, I also got a mail from the list llvm-commits: 'Your message entitled [PATCH] D21728: [AARCH64] Add Broadcom Vulcan scheduling modelwas successfully received by the llvm-commits mailing list.'
I can resend the patch, if it is required. Kindly suggest.
Thanks.

Jun 29 2016, 2:30 AM

Jun 25 2016

pgode retitled D21728: [AARCH64] Add Broadcom Vulcan scheduling model from to [AARCH64] Add Broadcom Vulcan scheduling model.
Jun 25 2016, 8:38 AM
pgode added a reviewer for D21501: [Driver] Add support for Broadcom Vulcan core: aemerson.
Jun 25 2016, 2:36 AM

Jun 23 2016

pgode added a comment to D21501: [Driver] Add support for Broadcom Vulcan core.

Ping ?

Jun 23 2016, 10:47 PM

Jun 22 2016

pgode added a reviewer for D21501: [Driver] Add support for Broadcom Vulcan core: rsmith.
Jun 22 2016, 5:12 AM

Jun 20 2016

pgode committed rL273148: [AARCH64] Add support for Broadcom Vulcan.
[AARCH64] Add support for Broadcom Vulcan
Jun 20 2016, 4:20 AM
pgode closed D21500: [AARCH64] Add support for Broadcom Vulcan by committing rL273148: [AARCH64] Add support for Broadcom Vulcan.
Jun 20 2016, 4:20 AM
pgode added inline comments to D21500: [AARCH64] Add support for Broadcom Vulcan.
Jun 20 2016, 3:17 AM

Jun 19 2016

pgode retitled D21501: [Driver] Add support for Broadcom Vulcan core from to [Driver] Add support for Broadcom Vulcan core.
Jun 19 2016, 1:57 AM
pgode retitled D21500: [AARCH64] Add support for Broadcom Vulcan from to [AARCH64] Add support for Broadcom Vulcan.
Jun 19 2016, 1:47 AM

Jun 15 2016

pgode committed rL272808: Test commit after access grant. Modified comment by adding a period. .
Test commit after access grant. Modified comment by adding a period.
Jun 15 2016, 10:31 AM