Page MenuHomePhabricator

jdd (John Demme)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 10 2020, 4:52 PM (76 w, 2 d)

Recent Activity

Thu, Nov 11

jdd added inline comments to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Thu, Nov 11, 12:21 PM · Restricted Project

Sep 22 2021

jdd added a comment to D110301: [MLIR] [Python] Make Attribute and Type hashable.

Thanks Stella!

Sep 22 2021, 8:03 PM · Restricted Project
jdd committed rG47cc166bc023: [MLIR] [Python] Make Attribute and Type hashable (authored by jdd).
[MLIR] [Python] Make Attribute and Type hashable
Sep 22 2021, 8:00 PM
jdd closed D110301: [MLIR] [Python] Make Attribute and Type hashable.
Sep 22 2021, 8:00 PM · Restricted Project
jdd requested review of D110301: [MLIR] [Python] Make Attribute and Type hashable.
Sep 22 2021, 6:47 PM · Restricted Project

Aug 20 2021

jdd accepted D108489: [mlir][python] Simplify python extension loading..

I'm probably not the best person to review this (since I'm not aware of the full history), but everything here looks reasonable to me and significantly less hacky to boot! Thanks

Aug 20 2021, 7:32 PM · Restricted Project

Aug 19 2021

jdd committed rG96fbd5cd5e44: [MLIR] [Python] Add `owner` to `mlir.ir.Block` (authored by jdd).
[MLIR] [Python] Add `owner` to `mlir.ir.Block`
Aug 19 2021, 12:13 AM

Aug 16 2021

jdd committed rG1689dade4218: [MLIR] [Python] Allow 'operation.parent' to return 'None' (authored by jdd).
[MLIR] [Python] Allow 'operation.parent' to return 'None'
Aug 16 2021, 10:38 PM
jdd closed D108183: [MLIR] [Python] Allow 'operation.parent' to return 'None'.
Aug 16 2021, 10:38 PM · Restricted Project
jdd updated the summary of D108183: [MLIR] [Python] Allow 'operation.parent' to return 'None'.
Aug 16 2021, 8:43 PM · Restricted Project
jdd updated the summary of D108183: [MLIR] [Python] Allow 'operation.parent' to return 'None'.
Aug 16 2021, 8:42 PM · Restricted Project
jdd requested review of D108183: [MLIR] [Python] Allow 'operation.parent' to return 'None'.
Aug 16 2021, 8:41 PM · Restricted Project
jdd added a comment to D108181: [MLIR] [Python] Fix out-of-tree Windows python bindings.

Landing this without review as it is "obvious".

Aug 16 2021, 7:20 PM · Restricted Project
jdd committed rG5821047aaca2: [MLIR] [Python] Fix out-of-tree Windows python bindings (authored by jdd).
[MLIR] [Python] Fix out-of-tree Windows python bindings
Aug 16 2021, 7:20 PM
jdd closed D108181: [MLIR] [Python] Fix out-of-tree Windows python bindings.
Aug 16 2021, 7:20 PM · Restricted Project
jdd added a reviewer for D108181: [MLIR] [Python] Fix out-of-tree Windows python bindings: stellaraccident.
Aug 16 2021, 7:17 PM · Restricted Project
jdd requested review of D108181: [MLIR] [Python] Fix out-of-tree Windows python bindings.
Aug 16 2021, 7:16 PM · Restricted Project

Jul 15 2021

jdd committed rGb4c93ece8e4f: [MLIR] [Python ODS] Use @builtins.property for cases where 'property' is… (authored by jdd).
[MLIR] [Python ODS] Use @builtins.property for cases where 'property' is…
Jul 15 2021, 7:40 PM
jdd closed D106106: [MLIR] [Python ODS] Use @builtins.property for cases where 'property' is already defined.
Jul 15 2021, 7:39 PM · Restricted Project
jdd added reviewers for D106106: [MLIR] [Python ODS] Use @builtins.property for cases where 'property' is already defined: stellaraccident, mikeurbach, mehdi_amini.
Jul 15 2021, 3:15 PM · Restricted Project
jdd requested review of D106106: [MLIR] [Python ODS] Use @builtins.property for cases where 'property' is already defined.
Jul 15 2021, 3:14 PM · Restricted Project

Jul 14 2021

jdd committed rG5664c5e24ed4: [MLIR] [Python] Add `owner` to PyValue and fix its parent reference (authored by jdd).
[MLIR] [Python] Add `owner` to PyValue and fix its parent reference
Jul 14 2021, 8:47 PM
jdd closed D103853: [MLIR] [Python] Add `owner` to PyValue and fix its parent reference.
Jul 14 2021, 8:46 PM · Restricted Project

Jun 7 2021

jdd retitled D103853: [MLIR] [Python] Add `owner` to PyValue and fix its parent reference from [MLIR] [Python] Fix parent of PyValue being constructed by PyOpOperandList to [MLIR] [Python] Add `owner` to PyValue and fix its parent reference.
Jun 7 2021, 4:44 PM · Restricted Project
jdd requested review of D103853: [MLIR] [Python] Add `owner` to PyValue and fix its parent reference.
Jun 7 2021, 4:40 PM · Restricted Project

May 23 2021

jdd committed rGc65bb760df1f: [MLIR] [Python] Add Operation.parent (authored by jdd).
[MLIR] [Python] Add Operation.parent
May 23 2021, 8:53 PM
jdd closed D102981: [MLIR] [Python] Add Operation.parent.
May 23 2021, 8:53 PM · Restricted Project
jdd updated the diff for D102981: [MLIR] [Python] Add Operation.parent.

Adding test

May 23 2021, 3:32 AM · Restricted Project
jdd added a reviewer for D102981: [MLIR] [Python] Add Operation.parent: stellaraccident.
May 23 2021, 3:06 AM · Restricted Project
jdd requested review of D102981: [MLIR] [Python] Add Operation.parent.
May 23 2021, 3:06 AM · Restricted Project

May 9 2021

jdd accepted D102144: [mlir][Python] Upstream the PybindAdaptors.h helpers and use it to implement sparse_tensor.encoding..

Psyched to see this upstream! We've been using a subset of this in CIRCT with only the issue Mike encountered, though that may have been us.

May 9 2021, 7:47 PM · Restricted Project

May 3 2021

jdd accepted D101734: [mlir][Python] Add casting constructor to Type and Attribute..

LGTM! Thanks

May 3 2021, 12:28 AM · Restricted Project

Apr 28 2021

jdd added a comment to D101496: [mlir] Move PyConcreteType to header. NFC..

Thanks @stellaraccident. So I assume you'd be opposed to making this a real public API by moving some of the python headers into 'mlir/include'? I was going to do that next since as of now, it's quite awkward to #include in out-of-tree code.

Apr 28 2021, 4:49 PM · Restricted Project
jdd committed rG32e2fec726be: [mlir] Move PyConcreteType to header. NFC. (authored by jdd).
[mlir] Move PyConcreteType to header. NFC.
Apr 28 2021, 4:46 PM
jdd closed D101496: [mlir] Move PyConcreteType to header. NFC..
Apr 28 2021, 4:46 PM · Restricted Project
jdd updated the summary of D101496: [mlir] Move PyConcreteType to header. NFC..
Apr 28 2021, 3:41 PM · Restricted Project
jdd requested review of D101496: [mlir] Move PyConcreteType to header. NFC..
Apr 28 2021, 3:40 PM · Restricted Project

Apr 22 2021

jdd added a comment to D101090: [MLIR][Python] Add capsule methods for pybind11 to PyValue..

Another thing we'll want is a nice casting helper, like this: https://github.com/llvm/circt/pull/940/commits/69f527952aa5acb2c73cc3b74e836483b5a5a5d8. It seems we have been adding this support downstream when needed. Does that kind of thing belong in the downstream project, or could I add it in MLIR? I don't see much pybind11 casting functionality in MLIR, so I'm curious if it's because it doesn't belong here or if we just haven't had the chance to move it upstream.

Apr 22 2021, 3:41 PM · Restricted Project
jdd added a comment to D101063: [mlir] Move PyConcreteAttribute to header. NFC..

This is great, though IRModule.h isn't in the mlir include directory, so it's kinda awkward for out-of-tree users to #include it. Could it be moved? (Along with a few other header files in the python directory.)

Apr 22 2021, 3:37 PM · Restricted Project

Apr 6 2021

jdd committed rG0126e906483c: [MLIR] [Python] Add capsule methods for pybind11 to PyOperation (authored by jdd).
[MLIR] [Python] Add capsule methods for pybind11 to PyOperation
Apr 6 2021, 2:30 PM
jdd closed D99927: [MLIR] [Python] Add capsule methods for pybind11 to PyOperation.
Apr 6 2021, 2:30 PM · Restricted Project
jdd added a comment to D99927: [MLIR] [Python] Add capsule methods for pybind11 to PyOperation.

Thanks! I think my success is more a testament to the ownership management infra already there. Landing this now.

Apr 6 2021, 2:23 PM · Restricted Project

Apr 5 2021

jdd published D99927: [MLIR] [Python] Add capsule methods for pybind11 to PyOperation for review.

@stellaraccident I really don't understand what I've done here, but it works... not that that means much when it comes to memory management. Am I doing it correctly or is there something I'm missing about Operation ownership semantics and interactions with PyObjectRef?

Apr 5 2021, 11:14 PM · Restricted Project

Feb 4 2021

jdd added a comment to D96093: [mlir] Introduce more intuitive wording for attributes..

“Intrinsic” makes me think of compiler intrinsics, which this is not. I like “property” since they are necessary properties of ops. I don’t have a strong preference other than the rejection reasons you give.

Feb 4 2021, 5:32 PM · Restricted Project

Jan 9 2021

jdd added a comment to D94274: [mlir][TypeDefGen] Add support for adding builders when generating a TypeDef.

Nice! Was this change being discussed in a bug which can now be closed? I seem to recall that but cannot find the bug.

Jan 9 2021, 1:04 PM · Restricted Project

Dec 15 2020

jdd added inline comments to D93270: [mlir][IR] Define the singleton builtin types in ODS instead of C++.
Dec 15 2020, 12:42 PM · Restricted Project
jdd accepted D93270: [mlir][IR] Define the singleton builtin types in ODS instead of C++.

This looks great!

Dec 15 2020, 12:41 PM · Restricted Project
jdd added inline comments to D93270: [mlir][IR] Define the singleton builtin types in ODS instead of C++.
Dec 15 2020, 9:13 AM · Restricted Project
jdd added inline comments to D93270: [mlir][IR] Define the singleton builtin types in ODS instead of C++.
Dec 15 2020, 12:24 AM · Restricted Project

Nov 22 2020

jdd committed rG95956c1c9aae: [MLIR] ODS typedef gen fixes & improvements (authored by jdd).
[MLIR] ODS typedef gen fixes & improvements
Nov 22 2020, 4:15 PM
jdd closed D91939: [MLIR] ODS typedef gen fixes & improvements.
Nov 22 2020, 4:15 PM · Restricted Project
jdd added a comment to D91939: [MLIR] ODS typedef gen fixes & improvements.

Nice, thanks! Does this add a verifier style check that makes sure that inner type is an RTLValueType, or is that still the responsibility of the custom parser logic?

Nov 22 2020, 3:57 PM · Restricted Project
jdd updated the summary of D91939: [MLIR] ODS typedef gen fixes & improvements.
Nov 22 2020, 3:34 PM · Restricted Project
jdd updated the summary of D91939: [MLIR] ODS typedef gen fixes & improvements.
Nov 22 2020, 3:30 PM · Restricted Project
jdd updated the summary of D91939: [MLIR] ODS typedef gen fixes & improvements.
Nov 22 2020, 3:30 PM · Restricted Project
jdd requested review of D91939: [MLIR] ODS typedef gen fixes & improvements.
Nov 22 2020, 3:29 PM · Restricted Project

Oct 27 2020

jdd committed rG035e12e66449: [MLIR] [ODS] Allowing attr-dict in custom directive (authored by jdd).
[MLIR] [ODS] Allowing attr-dict in custom directive
Oct 27 2020, 6:26 PM
jdd closed D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 27 2020, 6:25 PM · Restricted Project
jdd updated the diff for D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
  • const MutableDictionaryAttr&
Oct 27 2020, 4:43 PM · Restricted Project
jdd updated the diff for D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
  • Changes from feedback #2
Oct 27 2020, 4:26 PM · Restricted Project
jdd added inline comments to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 27 2020, 4:24 PM · Restricted Project

Oct 22 2020

jdd added a comment to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.

Not sure I'm interpreting all your comments correctly but it's certainly closer.

Oct 22 2020, 5:50 PM · Restricted Project
jdd updated the diff for D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
  • Adding test, making suggested changes, fixing breaks
Oct 22 2020, 5:47 PM · Restricted Project
jdd added inline comments to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 22 2020, 4:32 PM · Restricted Project
jdd added inline comments to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 22 2020, 1:35 PM · Restricted Project
jdd added inline comments to D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 22 2020, 1:29 PM · Restricted Project

Oct 20 2020

jdd requested review of D89772: [MLIR] [ODS] Allowing attr-dict in custom directive.
Oct 20 2020, 12:56 AM · Restricted Project

Oct 18 2020

jdd committed rGf402e682d0ef: [MLIR] ODS TypeDefs: getChecked() and internal enhancements (authored by jdd).
[MLIR] ODS TypeDefs: getChecked() and internal enhancements
Oct 18 2020, 6:11 PM
jdd closed D89438: [MLIR] ODS TypeDefs: getChecked() and internal enhancements.
Oct 18 2020, 6:10 PM · Restricted Project
jdd updated the diff for D89438: [MLIR] ODS TypeDefs: getChecked() and internal enhancements.
  • Updates based on feedback fro River
Oct 18 2020, 5:45 PM · Restricted Project
jdd committed rG6f878096640f: [MLIR] ODS TypeDef documentation (authored by jdd).
[MLIR] ODS TypeDef documentation
Oct 18 2020, 5:22 PM
jdd closed D89440: [MLIR] ODS TypeDef documentation.
Oct 18 2020, 5:22 PM · Restricted Project
jdd added inline comments to D89440: [MLIR] ODS TypeDef documentation.
Oct 18 2020, 5:18 PM · Restricted Project
jdd added inline comments to D89440: [MLIR] ODS TypeDef documentation.
Oct 18 2020, 5:01 PM · Restricted Project
jdd updated the diff for D89440: [MLIR] ODS TypeDef documentation.
  • Changes based on River's feedback.
Oct 18 2020, 5:01 PM · Restricted Project

Oct 15 2020

jdd added a comment to D89440: [MLIR] ODS TypeDef documentation.

Also, this documentation is very slightly wrong without https://reviews.llvm.org/D89438

Oct 15 2020, 4:51 PM · Restricted Project
jdd updated the diff for D89440: [MLIR] ODS TypeDef documentation.
  • Updating doc as per Mehdi's comments
Oct 15 2020, 4:48 PM · Restricted Project
jdd added inline comments to D89440: [MLIR] ODS TypeDef documentation.
Oct 15 2020, 4:46 PM · Restricted Project

Oct 14 2020

jdd added a comment to D89440: [MLIR] ODS TypeDef documentation.

I'm working on this in this branch: https://github.com/teqdruid/llvm-project/tree/ods-types-doc. It may be faster to edit directly. I can either push this branch upstream or give reviews/editors access to my fork.

Oct 14 2020, 7:26 PM · Restricted Project
jdd requested review of D89440: [MLIR] ODS TypeDef documentation.
Oct 14 2020, 7:23 PM · Restricted Project
jdd requested review of D89438: [MLIR] ODS TypeDefs: getChecked() and internal enhancements.
Oct 14 2020, 7:04 PM · Restricted Project

Oct 13 2020

jdd committed rG5fe53c412832: [MLIR] Add support for defining Types in tblgen (authored by jdd).
[MLIR] Add support for defining Types in tblgen
Oct 13 2020, 5:32 PM
jdd closed D86904: [mlir] Support for defining Types in tblgen.
Oct 13 2020, 5:32 PM · Restricted Project, Restricted Project
jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.
  • Rebasing w/ conflicts
  • New branch, applying everything at once
  • Clang-(format|tidy) fixes
  • One more clang-tidy fix
  • Changes based on Chris' comments
  • Ending a sentence comment w/ a period
  • Revisions base on River's comments
  • Clang-tidy and some missed comments
  • Missed a comment
  • clang-format
  • Changes based on review
  • Remove llvm:: from the cpp file
  • Clang-tidy fix
  • Another round of changes based on feedback.
Oct 13 2020, 3:00 PM · Restricted Project, Restricted Project
jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.
  • Another round of changes based on feedback.
Oct 13 2020, 2:15 PM · Restricted Project, Restricted Project
jdd added inline comments to D86904: [mlir] Support for defining Types in tblgen.
Oct 13 2020, 2:10 PM · Restricted Project, Restricted Project

Oct 10 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.
  • Clang-tidy fix
Oct 10 2020, 7:16 PM · Restricted Project, Restricted Project

Oct 9 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.
  • Changes based on review
  • Remove llvm:: from the cpp file
Oct 9 2020, 10:22 PM · Restricted Project, Restricted Project
jdd added a comment to D86904: [mlir] Support for defining Types in tblgen.

I'll update the documentation as you requested. Do you mind if I do it in a subsequent commit? I have a feeling there's going to be some back-and-forth on the documentation and I don't want that editorial process to gate this commit.

Oct 9 2020, 10:21 PM · Restricted Project, Restricted Project

Oct 8 2020

jdd added a comment to D89059: Add basic comparison operations for the RTL dialect.

Hi Andrew- We're using GH PRs to do reviews.

Oct 8 2020, 11:57 AM

Oct 5 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

clang-format

Oct 5 2020, 3:30 PM · Restricted Project, Restricted Project

Oct 2 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

Missed a comment

Oct 2 2020, 8:46 PM · Restricted Project, Restricted Project
jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

Revisions base on River's comments

Oct 2 2020, 8:43 PM · Restricted Project, Restricted Project
jdd updated the summary of D86904: [mlir] Support for defining Types in tblgen.
Oct 2 2020, 8:42 PM · Restricted Project, Restricted Project
jdd added inline comments to D86904: [mlir] Support for defining Types in tblgen.
Oct 2 2020, 7:19 PM · Restricted Project, Restricted Project
jdd added a comment to D86904: [mlir] Support for defining Types in tblgen.

As for declaring parse(...), print(...), and getMnemonic() in the header file: these are intended for cases where (for some reason) the dialect doesn't use the global parse/print dispatch method. The dialect could call each parse/print method in its Dialect::parseType/Dialect::printType methods, in concert with the getMnemonic(). I figure even if it's not often used, its just an extra 3 lines in each type declaration.

Oct 2 2020, 12:35 PM · Restricted Project, Restricted Project

Oct 1 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

Changes based on Chris' comments

Oct 1 2020, 11:16 AM · Restricted Project, Restricted Project

Sep 29 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

One more clang-tidy fix

Sep 29 2020, 11:54 AM · Restricted Project, Restricted Project

Sep 28 2020

jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

Fixing the previous arc diff

Sep 28 2020, 6:11 PM · Restricted Project, Restricted Project
jdd updated the diff for D86904: [mlir] Support for defining Types in tblgen.

Fixing whitespace and clang-tidy issues

Sep 28 2020, 6:10 PM · Restricted Project, Restricted Project