Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

pranavb (Pranav Bhandarkar)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 28 2017, 10:56 AM (326 w, 2 d)

Recent Activity

Jan 14 2022

pranavb added a comment to D117133: Fix Hexagon optimize addressing mode pass so that it handles only BaseImmOffset addressing mode..

Let me ping Krzysztof and Sumanth for a review of the latest revision. This is ready to land as far as I am concerned.

Jan 14 2022, 11:53 AM · Restricted Project

Jan 13 2022

pranavb updated the diff for D117133: Fix Hexagon optimize addressing mode pass so that it handles only BaseImmOffset addressing mode..

Fixed run line in testcase

Jan 13 2022, 2:39 PM · Restricted Project

Jan 12 2022

pranavb updated the diff for D117133: Fix Hexagon optimize addressing mode pass so that it handles only BaseImmOffset addressing mode..

Formatted using clang-format.

Jan 12 2022, 11:01 PM · Restricted Project
pranavb requested review of D117133: Fix Hexagon optimize addressing mode pass so that it handles only BaseImmOffset addressing mode..
Jan 12 2022, 10:59 AM · Restricted Project

Sep 15 2020

pranavb added a comment to rG0ee54cf88329: [Hexagon] Account for truncating pairs to non-pairs when widening truncates.

Thanks @kparzysz - that fixed it in my local repo as well!

Sep 15 2020, 9:16 PM
pranavb added a comment to rG0ee54cf88329: [Hexagon] Account for truncating pairs to non-pairs when widening truncates.
Sep 15 2020, 8:59 PM
pranavb added a comment to rG0ee54cf88329: [Hexagon] Account for truncating pairs to non-pairs when widening truncates.

Thanks @srj . I have been able to reproduce this internall on predicated_store_load.cpp. Ankit is looking into this.

Sep 15 2020, 5:05 PM

Aug 26 2020

pranavb requested review of D86661: Use properlyDominates in RDFLiveness when sorting on dominance..
Aug 26 2020, 1:55 PM · Restricted Project

Oct 8 2019

pranavb updated subscribers of rL371640: LiveIntervals: Split live intervals on multiple dead defs.

@kparzysz has posted an alternative here https://reviews.llvm.org/D68666

Oct 8 2019, 2:43 PM

Oct 7 2019

pranavb added a comment to rL371640: LiveIntervals: Split live intervals on multiple dead defs.

While this change is valid in terms of functionality, I think Krzysztof has a point here (http://lists.llvm.org/pipermail/llvm-dev/2019-October/135709.html). Simply computing the live interval for a virtual register should not be changing code. That concern should be handled separately.

Oct 7 2019, 2:44 PM
pranavb added a comment to rL371640: LiveIntervals: Split live intervals on multiple dead defs.

There is a bug in HexagonExpandCondsets being exposed by this (valid) change. We are working on a fix as we speak.

Oct 7 2019, 1:50 PM

Oct 4 2019

pranavb added a comment to rL371640: LiveIntervals: Split live intervals on multiple dead defs.

Update: This change is affecting the HexagonExpandCondsets pass. I hope to have more updates soon.

Oct 4 2019, 11:54 AM

Aug 27 2018

pranavb accepted D51167: [Pipeliner] Fix incorrect phi values in the epilog and kernel.
Aug 27 2018, 12:46 PM

Sep 22 2017

pranavb committed rL313993: Check vector elements for equivalence in the HexagonVectorLoopCarriedReuse pass.
Check vector elements for equivalence in the HexagonVectorLoopCarriedReuse pass
Sep 22 2017, 9:45 AM

Sep 21 2017

pranavb committed rL313936: [Hexagon] - Fix testcase for the HexagonVectorLoopCarriedReuse pass..
[Hexagon] - Fix testcase for the HexagonVectorLoopCarriedReuse pass.
Sep 21 2017, 4:13 PM
pranavb committed rL313926: Add a testfile that I missed in a previous commit that.
Add a testfile that I missed in a previous commit that
Sep 21 2017, 2:54 PM
pranavb committed rL313925: Enable the reuse of values computed in a previous loop iteration..
Enable the reuse of values computed in a previous loop iteration.
Sep 21 2017, 2:49 PM