Page MenuHomePhabricator

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

arpith-jacob (Arpith Jacob)
User

Projects

User does not belong to any projects.

User Details

User Since
Dec 8 2015, 9:32 PM (416 w, 3 d)

Recent Activity

Aug 31 2021

arpith-jacob accepted D109025: Support alias.scope and noalias metadata lowering on intrinsics..
Aug 31 2021, 11:43 PM · Restricted Project

Jul 21 2021

arpith-jacob accepted D106495: [mlir][LLVM] Revert bareptr calling convention handling as an argument materialization..
Jul 21 2021, 2:20 PM · Restricted Project

Jun 15 2021

arpith-jacob added a comment to D104280: Support lowering of index-cast on vector types..

Could you commit this patch? Thanks!

Jun 15 2021, 9:24 AM · Restricted Project
arpith-jacob updated the diff for D104280: Support lowering of index-cast on vector types..

Updated docs.

Jun 15 2021, 9:23 AM · Restricted Project

Jun 14 2021

arpith-jacob updated the summary of D104280: Support lowering of index-cast on vector types..
Jun 14 2021, 9:02 PM · Restricted Project
arpith-jacob requested review of D104280: Support lowering of index-cast on vector types..
Jun 14 2021, 9:01 PM · Restricted Project

Mar 11 2021

arpith-jacob added a comment to D98420: [mlir] Add LLVM loop codegen options to control software pipelining.

Thank you for the quick review. Please commit when you get the chance.

Mar 11 2021, 6:24 AM · Restricted Project, Restricted Project
arpith-jacob requested review of D98420: [mlir] Add LLVM loop codegen options to control software pipelining.
Mar 11 2021, 6:14 AM · Restricted Project, Restricted Project

Mar 9 2021

arpith-jacob accepted D98105: Rework LLVM Dialect LoopOptions attribute.
Mar 9 2021, 11:50 AM · Restricted Project

Mar 4 2021

arpith-jacob added inline comments to D97944: [mlir] Add an AccessGroup attribute to load/store LLVM dialect ops and generate the access_group LLVM metadata..
Mar 4 2021, 9:13 AM · Restricted Project
arpith-jacob updated the diff for D97944: [mlir] Add an AccessGroup attribute to load/store LLVM dialect ops and generate the access_group LLVM metadata..

Thank you for the reviews! Addressed comments.

Mar 4 2021, 9:11 AM · Restricted Project
arpith-jacob requested review of D97944: [mlir] Add an AccessGroup attribute to load/store LLVM dialect ops and generate the access_group LLVM metadata..
Mar 4 2021, 6:27 AM · Restricted Project

Mar 3 2021

arpith-jacob added inline comments to D96820: [mlir] Add loop codegen options to some LLVM dialect ops..
Mar 3 2021, 2:16 PM · Restricted Project
arpith-jacob added inline comments to D96820: [mlir] Add loop codegen options to some LLVM dialect ops..
Mar 3 2021, 11:14 AM · Restricted Project
arpith-jacob updated the diff for D96820: [mlir] Add loop codegen options to some LLVM dialect ops..

Replaced auto with type in some instances.

Mar 3 2021, 11:08 AM · Restricted Project
arpith-jacob updated the diff for D96820: [mlir] Add loop codegen options to some LLVM dialect ops..

Thanks for the review. Addressed all comments.

Mar 3 2021, 10:10 AM · Restricted Project

Mar 2 2021

arpith-jacob updated the diff for D96820: [mlir] Add loop codegen options to some LLVM dialect ops..

Updated implementation to use an llvm.metadata operation as suggested in
https://llvm.discourse.group/t/annotating-loops-with-codegen-options/2854

Mar 2 2021, 10:04 PM · Restricted Project

Feb 18 2021

arpith-jacob updated the diff for D96820: [mlir] Add loop codegen options to some LLVM dialect ops..

Addressed comments.

Feb 18 2021, 6:10 AM · Restricted Project

Feb 17 2021

arpith-jacob added a comment to D96820: [mlir] Add loop codegen options to some LLVM dialect ops..

Why are you modeling this with a string indirection to a dictionary attribute on the module itself instead of adding the attribute directly on every individual operations?

Feb 17 2021, 7:33 AM · Restricted Project

Feb 16 2021

arpith-jacob requested review of D96820: [mlir] Add loop codegen options to some LLVM dialect ops..
Feb 16 2021, 3:45 PM · Restricted Project

Aug 4 2020

arpith-jacob added a comment to D85048: [mlir] Conversion of ViewOp with memory space to LLVM..

Can someone with commit access land this patch? Thanks.

Aug 4 2020, 9:50 PM · Restricted Project
arpith-jacob retitled D85048: [mlir] Conversion of ViewOp with memory space to LLVM. from [mlir] Conversion of ViewOp with address space to LLVM. to [mlir] Conversion of ViewOp with memory space to LLVM..
Aug 4 2020, 1:58 PM · Restricted Project
arpith-jacob updated the diff for D85048: [mlir] Conversion of ViewOp with memory space to LLVM..

Rebase

Aug 4 2020, 11:53 AM · Restricted Project
arpith-jacob updated the diff for D85048: [mlir] Conversion of ViewOp with memory space to LLVM..

Rebase

Aug 4 2020, 11:37 AM · Restricted Project

Aug 3 2020

arpith-jacob updated the diff for D85048: [mlir] Conversion of ViewOp with memory space to LLVM..

Addressed comments. PTAL.

Aug 3 2020, 8:56 AM · Restricted Project
arpith-jacob retitled D85048: [mlir] Conversion of ViewOp with memory space to LLVM. from [mlir] Conversion of ViewOp with address space to LLVM. to [mlir] Conversion of ViewOp with memory space to LLVM..
Aug 3 2020, 8:55 AM · Restricted Project

Jul 31 2020

arpith-jacob added a reviewer for D85048: [mlir] Conversion of ViewOp with memory space to LLVM.: jpienaar.
Jul 31 2020, 2:55 PM · Restricted Project
arpith-jacob requested review of D85048: [mlir] Conversion of ViewOp with memory space to LLVM..
Jul 31 2020, 2:55 PM · Restricted Project

Jul 21 2020

arpith-jacob accepted D84039: [mlir] [VectorOps] Add scatter/gather operations to Vector dialect.
Jul 21 2020, 10:31 AM · Restricted Project

Nov 21 2017

arpith-jacob added inline comments to D14254: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget device RTLs..
Nov 21 2017, 10:57 AM · Restricted Project, Unknown Object (Project)
arpith-jacob added inline comments to D14254: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget device RTLs..
Nov 21 2017, 9:57 AM · Restricted Project, Unknown Object (Project)

Nov 9 2017

arpith-jacob added a comment to D39822: [NVPTX] Model (some) side effects of warp-synchronous data exchange intrinsics..

I was not sure if the *_sync intrinsics required preventing CSE since these intrinsics capture all state as arguments (lanes in a warp to sync as an argument). However, on Volta, I think different lanes in a warp can execute the intrinsic from different syntactic locations (i.e., different program counters). If true, then we do indeed have to model the data exchanged.

Nov 9 2017, 7:53 AM

Apr 21 2017

arpith-jacob added a comment to D32321: [OpenMP] Optimized default kernel launch parameters in CUDA plugin.

Hi Jonas,

Apr 21 2017, 5:12 PM · Unknown Object (Project)

Feb 16 2017

arpith-jacob added a comment to D30050: Bug 31948: Fix assertion when bitcasting constantexpr pointers.

Thank you for this. LGTM.

Feb 16 2017, 1:14 PM
arpith-jacob updated the diff for D29879: [OpenMP] Teams reduction on the NVPTX device..

Addressed review comments.

Feb 16 2017, 4:58 AM
arpith-jacob added a comment to D29879: [OpenMP] Teams reduction on the NVPTX device..

Alexey, do you any more comments on this patch?

Feb 16 2017, 3:06 AM

Feb 14 2017

arpith-jacob updated the diff for D29910: [OpenMP] Specialize default schedule on a worksharing loop on the NVPTX device..

Hi Alexey,

Feb 14 2017, 1:22 PM
arpith-jacob added a comment to D29879: [OpenMP] Teams reduction on the NVPTX device..

Alexey, thank you for your review. I have used SizeTy instead of assuming 64-bits.

Feb 14 2017, 11:28 AM
arpith-jacob updated the diff for D29879: [OpenMP] Teams reduction on the NVPTX device..

Use SizeTy instead of assuming 64 bits!

Feb 14 2017, 11:25 AM
arpith-jacob added a comment to D29910: [OpenMP] Specialize default schedule on a worksharing loop on the NVPTX device..

Hi Alexey,

Feb 14 2017, 5:00 AM

Feb 13 2017

arpith-jacob created D29910: [OpenMP] Specialize default schedule on a worksharing loop on the NVPTX device..
Feb 13 2017, 1:47 PM

Feb 12 2017

arpith-jacob created D29879: [OpenMP] Teams reduction on the NVPTX device..
Feb 12 2017, 4:29 PM
arpith-jacob abandoned D29506: [OpenMP] Teams reduction on the NVPTX device..
Feb 12 2017, 4:27 PM
arpith-jacob updated the diff for D29758: [OpenMP] Parallel reduction on the NVPTX device..

Minor fixup of comment style on emitInterWarpCopyFunction().

Feb 12 2017, 4:25 PM
arpith-jacob updated the diff for D29758: [OpenMP] Parallel reduction on the NVPTX device..

Updated patch to address Alexey's comments. Condensed parameters in emitReduction() to a struct Options.

Feb 12 2017, 12:58 PM

Feb 10 2017

arpith-jacob added inline comments to D29758: [OpenMP] Parallel reduction on the NVPTX device..
Feb 10 2017, 7:49 AM

Feb 9 2017

arpith-jacob created D29758: [OpenMP] Parallel reduction on the NVPTX device..
Feb 9 2017, 6:48 AM
arpith-jacob added a comment to D29506: [OpenMP] Teams reduction on the NVPTX device..

The patch is too big and quite hard to review? Could you split it into several smaller parts?

Feb 9 2017, 6:42 AM

Feb 3 2017

arpith-jacob added inline comments to D29506: [OpenMP] Teams reduction on the NVPTX device..
Feb 3 2017, 12:00 PM
arpith-jacob created D29506: [OpenMP] Teams reduction on the NVPTX device..
Feb 3 2017, 11:57 AM

Jan 27 2017

arpith-jacob updated the diff for D17657: [NVPTX] Add intrinsics to support named barriers..

Looks like this patch slipped through the cracks :( I've made the requested changes.

Jan 27 2017, 1:05 PM

Jan 25 2017

arpith-jacob created D29143: [OpenMP] Codegen support for 'target teams' on the NVPTX device..
Jan 25 2017, 12:44 PM
arpith-jacob created D29128: [OpenMP] Support for the proc_bind-clause on 'target parallel' on the NVPTX device..
Jan 25 2017, 8:16 AM

Jan 24 2017

arpith-jacob created D29087: [OpenMP] Support for thread_limit-clause on the 'target teams' directive..
Jan 24 2017, 8:55 AM
arpith-jacob created D29085: [OpenMP] Support for num_teams-clause on the 'target teams' directive..
Jan 24 2017, 7:51 AM
arpith-jacob created D29084: [OpenMP] Codegen support for 'target teams' on the host..
Jan 24 2017, 7:11 AM
arpith-jacob created D29083: [OpenMP] Support for the num_threads-clause on 'target parallel' on the NVPTX device..
Jan 24 2017, 6:59 AM
arpith-jacob created D29082: [OpenMP] Support for the num_threads-clause on 'target parallel'..
Jan 24 2017, 6:55 AM

Jan 23 2017

arpith-jacob created D29026: [OpenMP] DSAChecker bug fix for combined directives..
Jan 23 2017, 7:14 AM

Jan 18 2017

arpith-jacob added inline comments to D28753: [OpenMP] Codegen support for 'target parallel' on the host..
Jan 18 2017, 4:50 AM
arpith-jacob updated the diff for D28781: [OpenMP] Support for the if-clause on the combined directive 'target parallel'..

Inherit from OMPLexical scope with an added argument to reduce code duplication.

Jan 18 2017, 3:52 AM

Jan 17 2017

arpith-jacob added a comment to D28781: [OpenMP] Support for the if-clause on the combined directive 'target parallel'..

Another correction. We'll have to create a similar scope OMPTeamsScope that inherits from OMPLexicalScope for target-teams combined directives.

Jan 17 2017, 11:39 AM
arpith-jacob added inline comments to D28781: [OpenMP] Support for the if-clause on the combined directive 'target parallel'..
Jan 17 2017, 11:10 AM
arpith-jacob added inline comments to D28781: [OpenMP] Support for the if-clause on the combined directive 'target parallel'..
Jan 17 2017, 11:08 AM
arpith-jacob updated the diff for D28753: [OpenMP] Codegen support for 'target parallel' on the host..

The patch was updated to split 'emitParallelOrTeamsOutlinedFunction' into 'emitParallelOutlinedFunction' and 'emitTeamsOutlinedFunction' to enable the use of getCapturedStmt().

Jan 17 2017, 9:38 AM
arpith-jacob added inline comments to D28753: [OpenMP] Codegen support for 'target parallel' on the host..
Jan 17 2017, 8:20 AM

Jan 16 2017

arpith-jacob updated the diff for D28753: [OpenMP] Codegen support for 'target parallel' on the host..

Updated 'getOpenMPCaptureRegions' to return the OMPD_teams region kind for the teams directive.

Jan 16 2017, 9:20 PM
arpith-jacob updated the diff for D28753: [OpenMP] Codegen support for 'target parallel' on the host..

Added a method 'getCapturedStmt' as part of OMPExecutableDirective.

Jan 16 2017, 8:47 PM
arpith-jacob created D28781: [OpenMP] Support for the if-clause on the combined directive 'target parallel'..
Jan 16 2017, 2:28 PM
arpith-jacob added a comment to D28752: [OpenMP] Refactor code that calls codegen for target regions on the device..

Thanks Alexey.

Jan 16 2017, 7:35 AM

Jan 15 2017

arpith-jacob retitled D28755: [OpenMP] Codegen for the 'target parallel' directive on the NVPTX device. from to [OpenMP] Codegen for the 'target parallel' directive on the NVPTX device..
Jan 15 2017, 5:24 PM
arpith-jacob retitled D28753: [OpenMP] Codegen support for 'target parallel' on the host. from to [OpenMP] Codegen support for 'target parallel' on the host..
Jan 15 2017, 4:44 PM
arpith-jacob retitled D28752: [OpenMP] Refactor code that calls codegen for target regions on the device. from to [OpenMP] Refactor code that calls codegen for target regions on the device..
Jan 15 2017, 4:41 PM

Jan 9 2017

arpith-jacob updated the diff for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..

Use i1 type for bool after all. But this time use the api ConvertType().

Jan 9 2017, 9:33 AM
arpith-jacob updated the diff for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..

Using CGF.ConvertTypeForMem(Context.getBoolType()) to get the right type for 'bool' rather than using i1.

Jan 9 2017, 8:23 AM
arpith-jacob added inline comments to D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Jan 9 2017, 4:09 AM
arpith-jacob updated the diff for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..

Moved CommonActionTy to CGOpenMPRuntimeNVPTX.cpp and renamed it to NVPTXActionTy, allowing us to customize the class in the future, if necessary.

Jan 9 2017, 4:06 AM

Jan 5 2017

arpith-jacob removed a reviewer for D16460: Bug 10002 - [opencl] Wrongfully assuming RHS is always unsigned: arpith-jacob.
Jan 5 2017, 1:05 AM

Jan 3 2017

arpith-jacob updated the diff for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..

Updated patch based on reviews.

Jan 3 2017, 11:47 AM
arpith-jacob added inline comments to D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Jan 3 2017, 10:37 AM

Jan 2 2017

arpith-jacob added inline comments to D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Jan 2 2017, 4:38 AM

Dec 30 2016

arpith-jacob updated the diff for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..

Alexey, thank you for your review. I've updated the patch addressing your comments.

Dec 30 2016, 4:47 PM

Dec 29 2016

arpith-jacob added inline comments to D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Dec 29 2016, 2:24 PM
arpith-jacob added a reviewer for D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device.: gtbercea.
Dec 29 2016, 8:35 AM

Dec 28 2016

arpith-jacob added inline comments to D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Dec 28 2016, 3:18 PM
arpith-jacob retitled D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device. from to [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device..
Dec 28 2016, 3:15 PM
arpith-jacob updated the diff for D28125: [OpenMP] Update target codegen for NVPTX device..

Alexey and Justin, thank you for spending the time to review this patch. I've updated the patch accordingly. I've also removed a dot ('.') from the worker function name since the character is not accepted by the nvidia linker ptxas in function names.

Dec 28 2016, 2:35 PM
arpith-jacob updated the diff for D28124: [OpenMP] Code cleanup for NVPTX OpenMP codegen.

Addressed comments in review to start function names with a lowercase letter and to fix the enum type name along with the enumerator name.

Dec 28 2016, 2:19 PM

Dec 27 2016

arpith-jacob retitled D28125: [OpenMP] Update target codegen for NVPTX device. from to [OpenMP] Update target codegen for NVPTX device..
Dec 27 2016, 10:38 AM
arpith-jacob retitled D28124: [OpenMP] Code cleanup for NVPTX OpenMP codegen from to [OpenMP] Code cleanup for NVPTX OpenMP codegen.
Dec 27 2016, 8:53 AM

Mar 9 2016

arpith-jacob added inline comments to D17877: [OpenMP] Base support for target directive codegen on NVPTX device..
Mar 9 2016, 7:44 PM
arpith-jacob added inline comments to D17877: [OpenMP] Base support for target directive codegen on NVPTX device..
Mar 9 2016, 7:59 AM
arpith-jacob updated the diff for D17877: [OpenMP] Base support for target directive codegen on NVPTX device..

Stylistic changes to address feedback.

Mar 9 2016, 7:56 AM

Mar 4 2016

arpith-jacob added inline comments to D17877: [OpenMP] Base support for target directive codegen on NVPTX device..
Mar 4 2016, 2:02 PM
arpith-jacob updated the diff for D17877: [OpenMP] Base support for target directive codegen on NVPTX device..

Addressed feedback; see inline comments for details.

Mar 4 2016, 1:57 PM
arpith-jacob added a comment to D17890: [OpenMP][NVPTX][CUDA] Adding support for printf for an NVPTX OpenMP device..

Thanks for the quick review! The test cases are the same as the CUDA version so it should be fine.

Mar 4 2016, 10:30 AM
arpith-jacob retitled D17890: [OpenMP][NVPTX][CUDA] Adding support for printf for an NVPTX OpenMP device. from to [OpenMP][NVPTX][CUDA] Adding support for printf for an NVPTX OpenMP device..
Mar 4 2016, 8:44 AM

Mar 3 2016

arpith-jacob retitled D17877: [OpenMP] Base support for target directive codegen on NVPTX device. from to [OpenMP] Base support for target directive codegen on NVPTX device..
Mar 3 2016, 7:35 PM
arpith-jacob added a comment to D17738: [NVPTX] Fix function identifiers that are invalid in PTX and a bug fix for the case of name collisions..

The real stable solution I think is for the IR name to be the final
name. No further mangling involved.

Mar 3 2016, 2:01 PM
arpith-jacob added a comment to D17738: [NVPTX] Fix function identifiers that are invalid in PTX and a bug fix for the case of name collisions..

Friendly ping. Rafael, do you have further concerns? Thanks.

Mar 3 2016, 7:13 AM