Page MenuHomePhabricator

vpykhtin (Valery Pykhtin)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 28 2016, 8:30 AM (177 w, 4 d)

Recent Activity

Tue, Jun 18

vpykhtin committed rG7e854e1cdd23: [AMDGPU] Speed up live-in virtual register set computaion in… (authored by vpykhtin).
[AMDGPU] Speed up live-in virtual register set computaion in…
Tue, Jun 18, 4:41 AM
vpykhtin committed rL363661: [AMDGPU] Speed up live-in virtual register set computaion in….
[AMDGPU] Speed up live-in virtual register set computaion in…
Tue, Jun 18, 4:40 AM
vpykhtin closed D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive.
Tue, Jun 18, 4:40 AM · Restricted Project

Mon, Jun 17

vpykhtin committed rGbf3c59f79bcc: LiveInterval.h: add LiveRange::findIndexesLiveAt function - return a list of… (authored by vpykhtin).
LiveInterval.h: add LiveRange::findIndexesLiveAt function - return a list of…
Mon, Jun 17, 11:27 AM
vpykhtin committed rL363593: LiveInterval.h: add LiveRange::findIndexesLiveAt function - return a list of….
LiveInterval.h: add LiveRange::findIndexesLiveAt function - return a list of…
Mon, Jun 17, 11:27 AM
vpykhtin closed D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
Mon, Jun 17, 11:27 AM · Restricted Project

Fri, Jun 14

vpykhtin committed rGffeb01c113a5: [AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check (authored by vpykhtin).
[AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check
Fri, Jun 14, 9:35 AM
vpykhtin committed rL363418: [AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check.
[AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check
Fri, Jun 14, 9:34 AM
vpykhtin closed D63337: [AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check to prevent link failure.
Fri, Jun 14, 9:34 AM · Restricted Project
vpykhtin updated the diff for D63337: [AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check to prevent link failure.

updated description

Fri, Jun 14, 9:13 AM · Restricted Project
vpykhtin created D63337: [AMDGPU] Don't constrain callees with inlinehint from inlining on MaxBB check to prevent link failure.
Fri, Jun 14, 8:54 AM · Restricted Project

Fri, Jun 7

vpykhtin added a comment to D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..

ping

Fri, Jun 7, 11:24 AM · Restricted Project
vpykhtin committed rGcb8de55f4749: [AMDGPU] Constrain the AMDGPU inliner on maximum number of basic blocks in a… (authored by vpykhtin).
[AMDGPU] Constrain the AMDGPU inliner on maximum number of basic blocks in a…
Fri, Jun 7, 5:14 AM
vpykhtin committed rL362789: [AMDGPU] Constrain the AMDGPU inliner on maximum number of basic blocks in a….
[AMDGPU] Constrain the AMDGPU inliner on maximum number of basic blocks in a…
Fri, Jun 7, 5:14 AM
vpykhtin closed D62917: [AMDGPU] Constrain the inliner on maximum number of basic blocks in a caller function.
Fri, Jun 7, 5:14 AM · Restricted Project

Wed, Jun 5

vpykhtin updated the diff for D62917: [AMDGPU] Constrain the inliner on maximum number of basic blocks in a caller function.

updated diff with per review issues done.

Wed, Jun 5, 11:33 PM · Restricted Project
vpykhtin created D62917: [AMDGPU] Constrain the inliner on maximum number of basic blocks in a caller function.
Wed, Jun 5, 10:04 AM · Restricted Project

Fri, May 31

vpykhtin added a comment to D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..

Hi, this is the parent revision for the https://reviews.llvm.org/D62401.

Fri, May 31, 3:21 AM · Restricted Project

Wed, May 29

vpykhtin updated the diff for D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
  • fixed bug when an index was considered in-segment at the segment's end in some cases (lower_bound used instead of upper)
  • added fast (binary search) skip for non-containing segments
  • improved variable naming
Wed, May 29, 10:29 AM · Restricted Project

Tue, May 28

vpykhtin added a comment to D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..

this implementation contains bug with upper segment boundary and misses opportunity to search on sorted segments too, fixing.

Tue, May 28, 5:34 AM · Restricted Project

Mon, May 27

vpykhtin updated the diff for D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive.

replaced std::vector with SmallVector, moved out of the loop.

Mon, May 27, 9:02 AM · Restricted Project
vpykhtin updated the diff for D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
  • replaced output std::vector with template output iterator, return bool on found
  • added is_sorted assert
  • improved description
Mon, May 27, 9:02 AM · Restricted Project

May 24 2019

vpykhtin added inline comments to D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
May 24 2019, 11:22 AM · Restricted Project
vpykhtin added a reviewer for D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at.: arsenm.
May 24 2019, 11:01 AM · Restricted Project
vpykhtin added reviewers for D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at.: rampitec, qcolombet, MatzeB.
May 24 2019, 11:01 AM · Restricted Project
vpykhtin added a child revision for D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at.: D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive.
May 24 2019, 10:52 AM · Restricted Project
vpykhtin added a parent revision for D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive: D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
May 24 2019, 10:52 AM · Restricted Project
vpykhtin created D62411: LiveIntervals: add LiveRange::findIndexesLiveAt function - return a list of SlotIndexes the LiveRange live at..
May 24 2019, 10:52 AM · Restricted Project
vpykhtin updated the diff for D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive.

split LiveInterval.h change into different patch

May 24 2019, 10:49 AM · Restricted Project
vpykhtin created D62401: [AMDGPU] Speed up live-in virtual register set computaion in GCNScheduleDAGMILive.
May 24 2019, 9:06 AM · Restricted Project

Mar 14 2019

vpykhtin accepted D59191: [AMDGPU] Asm/disasm v_cndmask_b32_e64 with abs/neg source modifiers.

LGTM, Thanks!

Mar 14 2019, 6:08 AM · Restricted Project

Feb 12 2019

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

The DPP combiner pass is enabled since rL353691, https://reviews.llvm.org/rGded96df01e95

Feb 12 2019, 2:58 AM · Restricted Project, Restricted Project

Feb 11 2019

vpykhtin committed rGe1c338e5270a: [AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled… (authored by vpykhtin).
[AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled…
Feb 11 2019, 8:29 AM
vpykhtin committed rL353717: [AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled….
[AMDGPU] fix atomic_optimizations_buffer.ll test after DPP combiner was enabled…
Feb 11 2019, 8:28 AM
vpykhtin committed rGded96df01e95: [AMDGPU] Enable DPP combiner pass by default. (authored by vpykhtin).
[AMDGPU] Enable DPP combiner pass by default.
Feb 11 2019, 3:15 AM
vpykhtin committed rL353691: [AMDGPU] Enable DPP combiner pass by default..
[AMDGPU] Enable DPP combiner pass by default.
Feb 11 2019, 3:15 AM

Feb 8 2019

vpykhtin committed rG7fe97f8c7ca1: [AMDGPU] Fix DPP combiner (authored by vpykhtin).
[AMDGPU] Fix DPP combiner
Feb 8 2019, 4:03 AM
vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Thank you Connor! I really appreciate your effort on this DPP work.

Feb 8 2019, 4:03 AM · Restricted Project, Restricted Project
vpykhtin committed rL353513: [AMDGPU] Fix DPP combiner.
[AMDGPU] Fix DPP combiner
Feb 8 2019, 4:00 AM
vpykhtin closed D55444: AMDGPU: Fix DPP combiner.
Feb 8 2019, 3:59 AM · Restricted Project, Restricted Project

Feb 7 2019

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Thanks Nikolai!

Feb 7 2019, 3:45 AM · Restricted Project, Restricted Project
vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

rebased diff.

Feb 7 2019, 3:42 AM · Restricted Project, Restricted Project

Feb 4 2019

vpykhtin added a reviewer for D55444: AMDGPU: Fix DPP combiner: nhaehnle.
Feb 4 2019, 8:40 AM · Restricted Project, Restricted Project

Jan 24 2019

vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

Fixed issue with old = dpp src register when bound ctrl is off.

Jan 24 2019, 7:53 AM · Restricted Project, Restricted Project

Jan 21 2019

vpykhtin accepted D56923: [AMDGPU] Fixed hazard recognizer to walk predecessors.

LGTM, thanks!

Jan 21 2019, 9:03 AM
vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 21 2019, 6:42 AM · Restricted Project, Restricted Project
vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 21 2019, 6:34 AM · Restricted Project, Restricted Project

Jan 18 2019

vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 18 2019, 5:03 AM · Restricted Project, Restricted Project

Jan 16 2019

vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 16 2019, 6:11 AM · Restricted Project, Restricted Project

Jan 15 2019

vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 15 2019, 8:35 AM · Restricted Project, Restricted Project
vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 15 2019, 8:02 AM · Restricted Project, Restricted Project
vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Hi Nikolai,

Jan 15 2019, 5:48 AM · Restricted Project, Restricted Project
vpykhtin added inline comments to D55444: AMDGPU: Fix DPP combiner.
Jan 15 2019, 4:04 AM · Restricted Project, Restricted Project

Jan 14 2019

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

I think we reached the state this can be submitted (and probably enabled with subsequent patch). This would allow any of us make other fixes if required.

Jan 14 2019, 2:54 AM · Restricted Project, Restricted Project

Jan 11 2019

vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

Thanks! I wonder how easy is to get confused there. I updated diff with the latest found problems fixed.

Jan 11 2019, 3:59 AM · Restricted Project, Restricted Project

Jan 10 2019

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Anyways, the other cases look like maybe some other clever optimization for the immediate is hindering this one, for example this with signed minimum:

	v_bfrev_b32_e32 v8, -2                                        ; 7E1058C2
	s_nop 1                                                       ; BF800001
	v_mov_b32_dpp v8, v2  row_bcast:15 row_mask:0xa bank_mask:0xf ; 7E1002FA AF014202
	v_min_i32_e32 v2, v2, v8                                      ; 18041102

Maybe this pass needs to be moved earlier in the pipeline?

I'm not sure I can insert the pass that high, I'll think of how it can be skipped.

Jan 10 2019, 8:42 AM · Restricted Project, Restricted Project
vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Anyways, the other cases look like maybe some other clever optimization for the immediate is hindering this one, for example this with signed minimum:

	v_bfrev_b32_e32 v8, -2                                        ; 7E1058C2
	s_nop 1                                                       ; BF800001
	v_mov_b32_dpp v8, v2  row_bcast:15 row_mask:0xa bank_mask:0xf ; 7E1002FA AF014202
	v_min_i32_e32 v2, v2, v8                                      ; 18041102

Maybe this pass needs to be moved earlier in the pipeline?

Jan 10 2019, 8:18 AM · Restricted Project, Restricted Project
vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Thank you, Connor.

Jan 10 2019, 6:13 AM · Restricted Project, Restricted Project

Jan 9 2019

vpykhtin reopened D55444: AMDGPU: Fix DPP combiner.

reopening revision.

Jan 9 2019, 7:36 AM · Restricted Project, Restricted Project
vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

rebased diff.

Jan 9 2019, 7:30 AM · Restricted Project, Restricted Project
vpykhtin committed rL350730: Revert "[AMDGPU] Fix DPP combiner".
Revert "[AMDGPU] Fix DPP combiner"
Jan 9 2019, 7:25 AM
vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Hi Connor,

Jan 9 2019, 5:59 AM · Restricted Project, Restricted Project
vpykhtin committed rL350721: [AMDGPU] Fix DPP combiner.
[AMDGPU] Fix DPP combiner
Jan 9 2019, 5:49 AM
vpykhtin closed D55444: AMDGPU: Fix DPP combiner.
Jan 9 2019, 5:49 AM · Restricted Project, Restricted Project

Dec 21 2018

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

If there is no strong objections I would like to submit my latest patch here since the pass is disabled and the patch looks better anyway. Otherwise I'll return from NY holidays only on Jan 11.

Dec 21 2018, 9:22 AM · Restricted Project, Restricted Project

Dec 19 2018

vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

Fixed issue with identity values and other cases, f32/f16 identity values to be added later.

Dec 19 2018, 6:29 AM · Restricted Project, Restricted Project

Dec 13 2018

vpykhtin added a comment to D55444: AMDGPU: Fix DPP combiner.

Connor, indeed, my bad, I'll try to fix this in a couple of days.

Dec 13 2018, 4:13 AM · Restricted Project, Restricted Project

Dec 12 2018

vpykhtin updated the diff for D55444: AMDGPU: Fix DPP combiner.

updated diff, added tests.

Dec 12 2018, 9:30 AM · Restricted Project, Restricted Project

Dec 7 2018

vpykhtin created D55444: AMDGPU: Fix DPP combiner.
Dec 7 2018, 9:22 AM · Restricted Project, Restricted Project

Dec 6 2018

vpykhtin added a comment to D55314: AMDGPU: Turn on the DPP combiner by default.

Ok, thank you very much for the review and explanation, I'll try to address it shortly.

Dec 6 2018, 6:33 AM
vpykhtin committed rL348487: [AMDGPU] Partial revert of rL348371: Turn on the DPP combiner by default.
[AMDGPU] Partial revert of rL348371: Turn on the DPP combiner by default
Dec 6 2018, 6:24 AM
vpykhtin added a comment to D55314: AMDGPU: Turn on the DPP combiner by default.

Ok, I'll disable it. I'm not sure about 3rd point: are you sayng the pass doesn't actually perform the optimization or it's fundamentally wrong? Because it implemented to handle "identity" cases for add, mul and min/max.

Dec 6 2018, 6:18 AM
vpykhtin added a comment to D55314: AMDGPU: Turn on the DPP combiner by default.

Hi,

This change breaks most of the subgroups tests with RADV (ie. dEQP-VK.subgroups.arithmetic.*).

Any reasons why you enabled it by default? Looks like it now triggers a new bug in the AMDGPU backend.

Thanks!

Dec 6 2018, 5:07 AM

Dec 5 2018

vpykhtin committed rL348371: [AMDGPU]: Turn on the DPP combiner by default.
[AMDGPU]: Turn on the DPP combiner by default
Dec 5 2018, 7:24 AM
vpykhtin closed D55314: AMDGPU: Turn on the DPP combiner by default.
Dec 5 2018, 7:24 AM
vpykhtin updated the diff for D55314: AMDGPU: Turn on the DPP combiner by default.
Dec 5 2018, 6:59 AM
vpykhtin created D55314: AMDGPU: Turn on the DPP combiner by default.
Dec 5 2018, 4:13 AM

Nov 30 2018

vpykhtin committed rL347993: [AMDGPU] Combine DPP mov with use instructions (VOP1/2/3).
[AMDGPU] Combine DPP mov with use instructions (VOP1/2/3)
Nov 30 2018, 6:25 AM
vpykhtin closed D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).
Nov 30 2018, 6:25 AM · Restricted Project

Nov 8 2018

vpykhtin updated the diff for D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).

fixed per review issues:

Nov 8 2018, 8:09 AM · Restricted Project

Nov 2 2018

vpykhtin added inline comments to D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).
Nov 2 2018, 10:42 AM · Restricted Project
vpykhtin updated the diff for D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).

Fixed per review issues.

Nov 2 2018, 7:31 AM · Restricted Project

Oct 26 2018

vpykhtin added inline comments to D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).
Oct 26 2018, 10:45 AM · Restricted Project
vpykhtin added inline comments to D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).
Oct 26 2018, 10:13 AM · Restricted Project
vpykhtin created D53762: AMDGPU: Combine DPP mov with use instuctions (VOP1/2/3).
Oct 26 2018, 7:46 AM · Restricted Project

Oct 10 2018

vpykhtin committed rL344134: [TableGen] fix assert in !cast when used out of definition in a multiclass.
[TableGen] fix assert in !cast when used out of definition in a multiclass
Oct 10 2018, 3:55 AM
vpykhtin closed D53068: TableGen: fix assert in !cast when used out of definition in a multiclass.
Oct 10 2018, 3:54 AM
vpykhtin created D53068: TableGen: fix assert in !cast when used out of definition in a multiclass.
Oct 10 2018, 2:43 AM

Jun 4 2018

vpykhtin accepted D47664: [AMDGPU] Factored out common part of GCNRPTracker::reset().
Jun 4 2018, 7:09 AM
vpykhtin accepted D47661: [AMDGPU] Small refactoring in the scheduler.
Jun 4 2018, 7:07 AM

May 31 2018

vpykhtin accepted D47511: [AMDGPU] Construct memory clauses before RA.
May 31 2018, 2:04 AM

May 30 2018

vpykhtin added inline comments to D47511: [AMDGPU] Construct memory clauses before RA.
May 30 2018, 11:50 PM
vpykhtin accepted D47509: [AMDGPU] Track occupancy in MFI.
May 30 2018, 9:40 PM

May 25 2018

vpykhtin accepted D46992: [AMDGPU] Add perf hints to functions.
May 25 2018, 10:16 AM
vpykhtin added a comment to D46992: [AMDGPU] Add perf hints to functions.

LGTM. Just a hint: whenever you use "auto X = ..." it's worth to specify explicitly if X is pointer or reference. It's not only saves you from accidental temp object by copy but also makes program easier to read.

May 25 2018, 3:51 AM

Mar 5 2018

vpykhtin accepted D44020: [AMDGPU][MC][DOC] Updated AMD GPU assembler description.

I think it looks very good, thank you!

Mar 5 2018, 2:34 AM

Dec 27 2017

vpykhtin accepted D41598: [AMDGPU][MC] Incorrect parsing of flat/global atomic modifiers.

LGTM.

Dec 27 2017, 6:57 AM

Dec 5 2017

vpykhtin added a comment to D35267: Pass Divergence Analysis data to selection DAG to drive divergence dependent instruction selection.

In general adding "custom" code to SelectionDAGBuilder::setValue looks odd. Instead I would add a target-customizable postprocessing loop on pairs of Value <-> SDNode into SelectionDAGISel::SelectBasicBlock right after the DAG is created. The target hook should be able to get whatever it requires LLVM IR analisys and annotate SDNodes.

Dec 5 2017, 6:44 AM

Nov 20 2017

vpykhtin committed rL318649: AMDGPU: Partial ILP scheduler port from SelectionDAG to SchedulingDAG….
AMDGPU: Partial ILP scheduler port from SelectionDAG to SchedulingDAG…
Nov 20 2017, 6:38 AM
vpykhtin closed D39897: AMDGPU: Partial ILP scheduler port from SelectionDAG to SchedulingDAG (experimental) by committing rL318649: AMDGPU: Partial ILP scheduler port from SelectionDAG to SchedulingDAG….
Nov 20 2017, 6:38 AM