Page MenuHomePhabricator

RamNalamothu (Venkata Ramanaiah Nalamothu)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 25 2020, 6:50 PM (40 w, 1 d)

Recent Activity

Oct 27 2020

RamNalamothu updated the diff for D76884: [AMDGPU] Implement -amdgpu-spill-cfi-saved-regs.

Rebase onto LLVM master and address feedback

Oct 27 2020, 7:58 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76883: [AMDGPU] Implement CFI for CSR spills.

Rebase onto LLVM master and address feedback

Oct 27 2020, 7:56 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76882: [AMDGPU] Implement CFI for non-kernel functions.

Rebase onto LLVM master and address feedback

Oct 27 2020, 7:54 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76880: [AMDGPU] Emit entry function CFI.

Rebase onto LLVM master and address feedback

Oct 27 2020, 7:52 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76879: [AMDGPU] Begin emitting CFI for AMDGCN.

Rebase onto LLVM Master

Oct 27 2020, 7:49 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

Rebase onto LLVM Master

Oct 27 2020, 7:46 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging.

Rebase onto LLVM master

Oct 27 2020, 7:45 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging.

Rebase on top of LLVM master

Oct 27 2020, 7:43 AM · debug-info, Restricted Project

Oct 20 2020

RamNalamothu committed rG89f7ccea6f64: [AMDGPU] Remove getAllVGPR32() which cannot handle Accum VGPRs properly (authored by RamNalamothu).
[AMDGPU] Remove getAllVGPR32() which cannot handle Accum VGPRs properly
Oct 20 2020, 10:46 AM
RamNalamothu closed D89806: [AMDGPU] Remove getAllVGPR32() which cannot handle Accum VGPRs properly.
Oct 20 2020, 10:45 AM · Restricted Project
RamNalamothu requested review of D89806: [AMDGPU] Remove getAllVGPR32() which cannot handle Accum VGPRs properly.
Oct 20 2020, 9:42 AM · Restricted Project
RamNalamothu added inline comments to D76879: [AMDGPU] Begin emitting CFI for AMDGCN.
Oct 20 2020, 1:51 AM · debug-info, Restricted Project

Sep 29 2020

RamNalamothu added a comment to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

I haven't touched the streamer side of AsmPrinter in a long time, but...
On PS4 we default to -fno-exceptions and while the section ends up named .eh_frame instead of .debug_frame the content is fine; we've had no complaints from our debugger folks. Aside from the introduction of the .cfi directives (which you do want), I'm not aware that there's any codegen consequence to this combination.
So, I'm not clear why a special streamer is really necessary.

Sep 29 2020, 9:49 AM · debug-info, Restricted Project

Sep 10 2020

RamNalamothu added reviewers for D76884: [AMDGPU] Implement -amdgpu-spill-cfi-saved-regs: greened, cameron.mcinally.
Sep 10 2020, 4:26 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76884: [AMDGPU] Implement -amdgpu-spill-cfi-saved-regs.

Rebase onto LLVM master. Tidy up code little bit and couple of fixes.
And also, handle cases where no free VGPR is available to spill into and we have to spill into memeory.

Sep 10 2020, 4:25 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76883: [AMDGPU] Implement CFI for CSR spills: greened, cameron.mcinally.
Sep 10 2020, 4:04 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76883: [AMDGPU] Implement CFI for CSR spills.

Rebase onto LLVM master

Sep 10 2020, 4:04 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76882: [AMDGPU] Implement CFI for non-kernel functions: greened, cameron.mcinally.
Sep 10 2020, 3:59 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76882: [AMDGPU] Implement CFI for non-kernel functions.

Rebase onto LLVM master

Sep 10 2020, 3:58 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76880: [AMDGPU] Emit entry function CFI: greened, cameron.mcinally.
Sep 10 2020, 3:55 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76880: [AMDGPU] Emit entry function CFI.

Rebase onto LLVM master

Sep 10 2020, 3:54 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76879: [AMDGPU] Begin emitting CFI for AMDGCN: greened, cameron.mcinally.
Sep 10 2020, 3:51 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76879: [AMDGPU] Begin emitting CFI for AMDGCN.

Rebase onto LLVM master

Sep 10 2020, 3:51 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging: greened, cameron.mcinally.
Sep 10 2020, 3:46 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging.

Rebase onto LLVM master

Sep 10 2020, 3:45 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo: greened, cameron.mcinally.
Sep 10 2020, 3:41 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

Rebase onto LLVM master

Sep 10 2020, 3:40 AM · debug-info, Restricted Project
RamNalamothu added reviewers for D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging: greened, cameron.mcinally.
Sep 10 2020, 3:38 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging.

Rebase onto LLVM master

Sep 10 2020, 3:36 AM · debug-info, Restricted Project

Sep 5 2020

RamNalamothu committed rGaff94ec0f4de: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory (authored by RamNalamothu).
[AMDGPU] Remove the dead spill slots while spilling FP/BP to memory
Sep 5 2020, 6:35 PM
RamNalamothu closed D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.
Sep 5 2020, 6:35 PM · Restricted Project

Sep 2 2020

RamNalamothu added inline comments to D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.
Sep 2 2020, 1:54 PM · Restricted Project
RamNalamothu updated the diff for D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.

Ah, wrong changset was committed in the previous update.

Sep 2 2020, 1:51 PM · Restricted Project
RamNalamothu updated the diff for D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.

Address review comments.

Sep 2 2020, 1:46 PM · Restricted Project
RamNalamothu updated the diff for D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.

Update a relevant test in 'callee-frame-setup.ll' for validating the patch.

Sep 2 2020, 8:56 AM · Restricted Project
RamNalamothu requested review of D87032: [AMDGPU] Remove the dead spill slots while spilling FP/BP to memory.
Sep 2 2020, 8:23 AM · Restricted Project

Aug 24 2020

RamNalamothu committed rGb9496efbb9eb: [AMDGPU, docs] Fix typos (authored by RamNalamothu).
[AMDGPU, docs] Fix typos
Aug 24 2020, 11:31 AM
RamNalamothu closed D86340: [AMDGPU, docs] Fix typos.
Aug 24 2020, 11:31 AM · Restricted Project
RamNalamothu retitled D86340: [AMDGPU, docs] Fix typos from [AMDGPU, docs] Test commit access to [AMDGPU, docs] Fix typos.
Aug 24 2020, 11:17 AM · Restricted Project
RamNalamothu updated the diff for D86340: [AMDGPU, docs] Fix typos.

Another attempt to update title. New to arcanist :)

Aug 24 2020, 2:17 AM · Restricted Project
RamNalamothu updated the diff for D86340: [AMDGPU, docs] Fix typos.

Update the title and address review comments.

Aug 24 2020, 1:32 AM · Restricted Project
RamNalamothu added inline comments to D86340: [AMDGPU, docs] Fix typos.
Aug 24 2020, 12:40 AM · Restricted Project

Aug 21 2020

RamNalamothu added reviewers for D86340: [AMDGPU, docs] Fix typos: t-tye, scott.linder.
Aug 21 2020, 4:29 AM · Restricted Project
RamNalamothu requested review of D86340: [AMDGPU, docs] Fix typos.
Aug 21 2020, 4:28 AM · Restricted Project

Aug 20 2020

RamNalamothu added reviewers for D86291: [AMDGPU] allSGPRSpillsAreDead() should use actual FP/BP frame indices: arsenm, scott.linder.
Aug 20 2020, 7:48 AM · Restricted Project
RamNalamothu requested review of D86291: [AMDGPU] allSGPRSpillsAreDead() should use actual FP/BP frame indices.
Aug 20 2020, 7:46 AM · Restricted Project

Jun 26 2020

RamNalamothu updated the diff for D76884: [AMDGPU] Implement -amdgpu-spill-cfi-saved-regs.

Rebase

Jun 26 2020, 5:24 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76883: [AMDGPU] Implement CFI for CSR spills.

Rebase and update get{S,V}GPRSpillSaveOpcode (SIInstrInfo.cpp) as
per https://reviews.llvm.org/rG46579471fd2b9c00ba982325f53f30192cc5797f

Jun 26 2020, 5:24 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76882: [AMDGPU] Implement CFI for non-kernel functions.

Rebase

Jun 26 2020, 5:24 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76880: [AMDGPU] Emit entry function CFI.

Rebase

Jun 26 2020, 5:24 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76879: [AMDGPU] Begin emitting CFI for AMDGCN.

Rebase

Jun 26 2020, 4:54 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging.

Rebase

Jun 26 2020, 4:51 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

Rebase

Jun 26 2020, 4:50 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging.

Rebase and update CFIProgram::parse (DWARFDebugFrame.cpp) changes as
per https://reviews.llvm.org/rG1e820e82b1438a52124512175a0e7c6f8d23e158

Jun 26 2020, 4:50 AM · debug-info, Restricted Project

Jun 11 2020

RamNalamothu updated the diff for D76883: [AMDGPU] Implement CFI for CSR spills.

Rebase

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76884: [AMDGPU] Implement -amdgpu-spill-cfi-saved-regs.

Rebase and fix position of spill for -amdgpu-spill-cfi-saved-regs

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D79878: [NFC] Move getAll{S,V}GPR{32,128} methods to SIFrameLowering.

Rebase

Jun 11 2020, 3:01 AM · Restricted Project
RamNalamothu updated the diff for D76882: [AMDGPU] Implement CFI for non-kernel functions.

Rebase and emit CFI for Base pointer

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76881: [AMDGPU] Skip CFIInstructions in SIInsertWaitcnts.

Rebase

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76880: [AMDGPU] Emit entry function CFI.

Rebase and add an MIR test

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76879: [AMDGPU] Begin emitting CFI for AMDGCN.

Rebase and update tests

Jun 11 2020, 3:01 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76878: Implement DW_{OP,AT}_LLVM_* for Heterogeneous Debugging.

Rebase

Jun 11 2020, 2:26 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

Rebase and address feedback

Jun 11 2020, 2:26 AM · debug-info, Restricted Project
RamNalamothu updated the diff for D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging.

Rebase and address feedback.

Jun 11 2020, 2:26 AM · debug-info, Restricted Project

May 20 2020

RamNalamothu added inline comments to D76880: [AMDGPU] Emit entry function CFI.
May 20 2020, 2:07 AM · debug-info, Restricted Project

May 19 2020

Herald added a reviewer for D67469: [WIP][Debuginfo][LLD] Remove obsolete debug info while garbage collecting.: MaskRay.
May 19 2020, 8:54 PM · debug-info, lld, Restricted Project

May 13 2020

RamNalamothu added inline comments to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.
May 13 2020, 6:34 PM · debug-info, Restricted Project

May 12 2020

RamNalamothu added inline comments to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.
May 12 2020, 7:56 PM · debug-info, Restricted Project
RamNalamothu added inline comments to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.
May 12 2020, 7:24 PM · debug-info, Restricted Project

May 11 2020

RamNalamothu added inline comments to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.
May 11 2020, 9:35 PM · debug-info, Restricted Project
RamNalamothu accepted D76879: [AMDGPU] Begin emitting CFI for AMDGCN.

LGTM

May 11 2020, 9:07 AM · debug-info, Restricted Project
RamNalamothu added a comment to D78778: [AMDGPU] Add SupportsDebugUnwindInformation to MCAsmInfo.

@scott.linder is this not yet ready for review, as you have not added any reviewers?

May 11 2020, 6:56 AM · debug-info, Restricted Project

May 5 2020

RamNalamothu added inline comments to D76877: Implement DW_CFA_LLVM_* for Heterogeneous Debugging.
May 5 2020, 6:58 AM · debug-info, Restricted Project

May 4 2020

RamNalamothu added inline comments to D79073: [AMDGPU] For PAL, make sure Scratch Buffer Descriptor do not clobber GIT pointer.
May 4 2020, 9:37 AM · Restricted Project
RamNalamothu updated the diff for D79073: [AMDGPU] For PAL, make sure Scratch Buffer Descriptor do not clobber GIT pointer.

Rebase and address feedback.

May 4 2020, 9:05 AM · Restricted Project

Apr 30 2020

RamNalamothu updated the diff for D79073: [AMDGPU] For PAL, make sure Scratch Buffer Descriptor do not clobber GIT pointer.

Address review comments.

Apr 30 2020, 11:47 AM · Restricted Project

Apr 29 2020

RamNalamothu created D79073: [AMDGPU] For PAL, make sure Scratch Buffer Descriptor do not clobber GIT pointer.
Apr 29 2020, 2:06 AM · Restricted Project

Mar 26 2020

RamNalamothu accepted D76732: [AMDGPU] Fix PC register mapping in wave32 mode.

Sorry for missing this in the initial review, I caught it as I was rebasing my CFI changes and adding tests for wave32.

Mar 26 2020, 12:30 AM · Restricted Project

Mar 23 2020

RamNalamothu added a comment to rG24698e526f61: Implement wave32 DWARF register mapping.

llvm/unittests/MC/AMDGPU/DwarfRegMappings.cpp seems to duplicate llvm/unittests/Target/AMDGPU/DwarfRegMappings.cpp in many lines. Is that intended?

Mar 23 2020, 10:49 PM

Mar 19 2020

RamNalamothu added inline comments to D76357: [AMDGPU] Implement wave32 DWARF register mapping.
Mar 19 2020, 12:32 PM · Restricted Project
RamNalamothu updated the diff for D76357: [AMDGPU] Implement wave32 DWARF register mapping.

Use array for test values and eliminate memory leak.

Mar 19 2020, 12:32 PM · Restricted Project
RamNalamothu updated the diff for D76357: [AMDGPU] Implement wave32 DWARF register mapping.

Address review comments.

Mar 19 2020, 1:03 AM · Restricted Project

Mar 18 2020

RamNalamothu added inline comments to D76357: [AMDGPU] Implement wave32 DWARF register mapping.
Mar 18 2020, 9:14 AM · Restricted Project
RamNalamothu updated the diff for D76357: [AMDGPU] Implement wave32 DWARF register mapping.

Address review comments.

Mar 18 2020, 9:14 AM · Restricted Project
RamNalamothu created D76357: [AMDGPU] Implement wave32 DWARF register mapping.
Mar 18 2020, 6:30 AM · Restricted Project