Page MenuHomePhabricator
People jeanPerier

jeanPerier (Jean Perier)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 16 2020, 6:35 AM (66 w, 14 h)

Recent Activity
View All

Today

jeanPerier accepted D104562: [flang] Fold more reduction intrinsic function calls.

Looks good

Mon, Jun 21, 1:31 AM · Restricted Project, Restricted Project
jeanPerier added a comment to D104586: [flang] Update Reshape runtime library routine and its corresponding unit test.

Looks good to me.

Mon, Jun 21, 1:19 AM · Restricted Project

Fri, Jun 18

jeanPerier accepted D104485: [flang] Runtime implementation for default derived type formatted I/O.

LGTM

Fri, Jun 18, 5:27 AM · Restricted Project, Restricted Project

Thu, Jun 17

jeanPerier committed rG1a4af2e45ee5: [flang] preserve symbol in DescriptorInquiry (authored by jeanPerier).
[flang] preserve symbol in DescriptorInquiry
Thu, Jun 17, 3:42 AM
jeanPerier closed D104385: [flang] preserve symbol in DescriptorInquiry.
Thu, Jun 17, 3:42 AM · Restricted Project, Restricted Project

Wed, Jun 16

jeanPerier requested review of D104385: [flang] preserve symbol in DescriptorInquiry.
Wed, Jun 16, 7:47 AM · Restricted Project, Restricted Project
jeanPerier accepted D104337: [flang] Fold MAXVAL & MINVAL.
Wed, Jun 16, 2:46 AM · Restricted Project, Restricted Project
jeanPerier accepted D104339: [flang] Implement runtime for IALL & IANY.

LGTM

Wed, Jun 16, 2:39 AM · Restricted Project, Restricted Project
jeanPerier accepted D104338: [flang] Use a "double-double" accumulator in SUM.

LGTM

Wed, Jun 16, 2:33 AM · Restricted Project, Restricted Project
jeanPerier accepted D104336: [flang] Cope with errors with array constructors.

LGTM

Wed, Jun 16, 2:31 AM · Restricted Project, Restricted Project

Tue, Jun 15

jeanPerier accepted D104290: [flang] Check there's no dependency on C++ libs.

Looks like a good idea to me to have this test, thanks !

Tue, Jun 15, 8:04 AM · Restricted Project, Restricted Project

Mon, Jun 14

jeanPerier accepted D104100: [flang] Add clang-tidy check for braces around if.

tco, fir-opt and bbc tools are lowering tools, so they currently use llvm style.

Mon, Jun 14, 7:25 AM · Restricted Project, Restricted Project

Fri, Jun 11

jeanPerier accepted D104019: [flang] Add initial implementation for CPU_TIME.

LGTM, thanks !

Fri, Jun 11, 1:28 AM · Restricted Project, Restricted Project

Tue, Jun 8

jeanPerier accepted D103805: [flang] Define the runtime API for CPU_TIME.

LGTM

Tue, Jun 8, 2:00 AM · Restricted Project, Restricted Project

Fri, Jun 4

jeanPerier accepted D103654: [flang] CheckConformance: tristate-ify result.

LGTM

Fri, Jun 4, 5:40 AM · Restricted Project, Restricted Project
jeanPerier committed rG1971960a6fed: [flang] Generate type info symbols outside of derived type scopes (authored by jeanPerier).
[flang] Generate type info symbols outside of derived type scopes
Fri, Jun 4, 5:39 AM
jeanPerier closed D103621: [flang] Generate type info symbols outside of derived type scopes.
Fri, Jun 4, 5:39 AM · Restricted Project, Restricted Project

Thu, Jun 3

jeanPerier requested review of D103621: [flang] Generate type info symbols outside of derived type scopes.
Thu, Jun 3, 8:31 AM · Restricted Project, Restricted Project
jeanPerier accepted D103580: [flang] Fix spurious "already declared" errors for interfaces.
Thu, Jun 3, 6:04 AM · Restricted Project, Restricted Project
jeanPerier accepted D103571: [flang] Support known constant lengths in DynamicType.

Looks good to me

Thu, Jun 3, 6:01 AM · Restricted Project, Restricted Project
jeanPerier accepted D103560: [flang] Check for duplicate definitions of defined input/output procedures.

Looks good to me

Thu, Jun 3, 5:56 AM · Restricted Project, Restricted Project
jeanPerier accepted D103568: [flang] Fix folding of CMPLX.
Thu, Jun 3, 5:49 AM · Restricted Project, Restricted Project

Tue, May 25

jeanPerier accepted D103098: [flang] Fix recent regression (proc. dummy arg on ENTRY).

LGTM, thanks

Tue, May 25, 9:32 AM · Restricted Project, Restricted Project

May 20 2021

jeanPerier committed rG943839870a0b: [flang] simplify derived type info table format (authored by jeanPerier).
[flang] simplify derived type info table format
May 20 2021, 9:28 AM
jeanPerier closed D102768: [flang] simplify derived type info table format.
May 20 2021, 9:28 AM · Restricted Project, Restricted Project
jeanPerier updated the diff for D102768: [flang] simplify derived type info table format.

DEREF only the false part of introduced ternary.

May 20 2021, 9:22 AM · Restricted Project, Restricted Project

May 19 2021

jeanPerier requested review of D102768: [flang] simplify derived type info table format.
May 19 2021, 6:25 AM · Restricted Project, Restricted Project

May 17 2021

jeanPerier accepted D102652: [flang] Implement MATMUL in the runtime.

LGTM

May 17 2021, 11:51 PM · Restricted Project, Restricted Project

May 13 2021

jeanPerier accepted D102351: [flang] Implement DOT_PRODUCT in the runtime.

LGTM

May 13 2021, 6:25 AM · Restricted Project, Restricted Project

May 11 2021

jeanPerier accepted D102210: [flang] Allow large and erroneous ac-implied-do's.
May 11 2021, 4:36 AM · Restricted Project, Restricted Project

May 6 2021

jeanPerier accepted D101928: [flang] Fix race condition in runtime.

The fix looks in line with the issue reported by @rogfer01 and what he suggested.

May 6 2021, 1:23 AM · Restricted Project, Restricted Project
jeanPerier accepted D101929: [flang] Runtime must defer formatted/unformatted determination.

LGTM

May 6 2021, 12:48 AM · Restricted Project, Restricted Project

Apr 19 2021

jeanPerier accepted D100599: [flang] Define missing & needed IEEE_ARITHMETIC symbols.

LGTM

Apr 19 2021, 8:36 AM · Restricted Project, Restricted Project

Apr 13 2021

jeanPerier accepted D100421: [flang] Handle END= situations better in runtime input.
Apr 13 2021, 11:58 PM · Restricted Project, Restricted Project
jeanPerier accepted D100419: [flang] Fix Boolean flag arguments.
Apr 13 2021, 11:53 PM · Restricted Project, Restricted Project

Apr 12 2021

jeanPerier accepted D100323: [flang] Correct TypeCode::IsLogical().

LGTM

Apr 12 2021, 10:52 AM · Restricted Project, Restricted Project

Apr 9 2021

jeanPerier accepted D100064: [flang] RANDOM_NUMBER, RANDOM_SEED, RANDOM_INIT in runtime.

LGTM

Apr 9 2021, 12:33 AM · Restricted Project, Restricted Project

Apr 8 2021

jeanPerier accepted D100067: [flang] Check for conflicting BIND(C) names.

LGTM

Apr 8 2021, 12:38 AM · Restricted Project, Restricted Project
jeanPerier accepted D100065: [flang] Enforce a limit on recursive PDT instantiations.

LGTM

Apr 8 2021, 12:12 AM · Restricted Project, Restricted Project

Apr 7 2021

jeanPerier accepted D99967: [Flang] Changes to mangling code.
Apr 7 2021, 2:21 AM · Restricted Project

Apr 1 2021

jeanPerier accepted D99666: [flang] Implement reductions in the runtime.

LGTM, thanks for implementing all these so fast !

Apr 1 2021, 1:40 AM · Restricted Project, Restricted Project

Mar 25 2021

jeanPerier committed rG27899112c698: [flang] fold LOGICAL intrinsic calls (authored by jeanPerier).
[flang] fold LOGICAL intrinsic calls
Mar 25 2021, 10:39 AM
jeanPerier closed D99346: [flang] fold LOGICAL intrinsic calls.
Mar 25 2021, 10:38 AM · Restricted Project, Restricted Project
jeanPerier requested review of D99346: [flang] fold LOGICAL intrinsic calls.
Mar 25 2021, 8:40 AM · Restricted Project, Restricted Project
jeanPerier committed rGffa455d4d451: [mlir] Translate global initializers after creating all LLVM IR globals (authored by jeanPerier).
[mlir] Translate global initializers after creating all LLVM IR globals
Mar 25 2021, 1:54 AM
jeanPerier closed D99246: [mlir] Translate global initializers after creating all LLVM IR globals.
Mar 25 2021, 1:54 AM · Restricted Project
jeanPerier added inline comments to D99246: [mlir] Translate global initializers after creating all LLVM IR globals.
Mar 25 2021, 1:39 AM · Restricted Project
jeanPerier updated the diff for D99246: [mlir] Translate global initializers after creating all LLVM IR globals.

Rename dropGlobalInitializer to shouldDropGlobalInitializer.

Mar 25 2021, 1:38 AM · Restricted Project
jeanPerier added a comment to D99296: [flang] Fix runtime crash on TRIM('').

Regarding CFI_establish, the Fortran standard says in 18.5.5.5 about elem_len :

Mar 25 2021, 1:19 AM · Restricted Project, Restricted Project

Mar 24 2021

jeanPerier requested review of D99246: [mlir] Translate global initializers after creating all LLVM IR globals.
Mar 24 2021, 2:04 AM · Restricted Project
jeanPerier accepted D99208: [flang] Save binding labels as strings.

LGTM, thanks that will make the lowering part cleaner.

Mar 24 2021, 1:33 AM · Restricted Project, Restricted Project

Mar 16 2021

jeanPerier committed rG92d27b969ae1: [flang] Save AllocateObject and PointerObject analyzed expression (authored by jeanPerier).
[flang] Save AllocateObject and PointerObject analyzed expression
Mar 16 2021, 2:30 AM
jeanPerier closed D98256: [flang] Save AllocateObject and PointerObject analyzed expression.
Mar 16 2021, 2:30 AM · Restricted Project, Restricted Project

Mar 15 2021

jeanPerier added inline comments to D98256: [flang] Save AllocateObject and PointerObject analyzed expression.
Mar 15 2021, 8:02 AM · Restricted Project, Restricted Project
jeanPerier accepted D98553: [flang] Runtime: implement INDEX intrinsic function.

LGTM

Mar 15 2021, 1:16 AM · Restricted Project, Restricted Project

Mar 11 2021

jeanPerier updated the summary of D98256: [flang] Save AllocateObject and PointerObject analyzed expression.
Mar 11 2021, 4:51 AM · Restricted Project, Restricted Project
jeanPerier updated the diff for D98256: [flang] Save AllocateObject and PointerObject analyzed expression.

Move the typedExpr setting in expression analysis. Update dump and unparse to use
the typed expression. Add a HasTypeExpr<T> helper to better share the logic when that
is possible and easy.

Mar 11 2021, 4:46 AM · Restricted Project, Restricted Project

Mar 10 2021

jeanPerier accepted D98283: [flang][fir] Upstream changes to the pre-FIR tree.

You may want to apply the clang-tidy hints about const auto ([llvm-qualified-auto]), otherwise LGTM and builds ok for me.

Mar 10 2021, 5:36 AM · Restricted Project, Restricted Project

Mar 9 2021

jeanPerier added a comment to D98256: [flang] Save AllocateObject and PointerObject analyzed expression.
In D98256#2614130, @PeteSteinfeld wrote:
Mar 9 2021, 9:58 AM · Restricted Project, Restricted Project
jeanPerier requested review of D98256: [flang] Save AllocateObject and PointerObject analyzed expression.
Mar 9 2021, 6:16 AM · Restricted Project, Restricted Project

Mar 4 2021

jeanPerier accepted D97996: [flang][fir] Add diagnostic tests for FIR ops verifier.

Thanks !

Mar 4 2021, 10:21 PM · Restricted Project

Mar 1 2021

jeanPerier accepted D97580: [flang] Runtime: SCAN and VERIFY.

Logic looks good to me. Please take care of the Pre-merge checks if they make sense.

Mar 1 2021, 12:40 AM · Restricted Project, Restricted Project

Feb 25 2021

jeanPerier accepted D97421: [flang][fir] Add array value operations..

Looks good to me.

Feb 25 2021, 12:13 PM · Restricted Project, Restricted Project

Feb 24 2021

jeanPerier committed rGb146dfe527ba: [flang] add attribute to trim runtime implementation establish call (authored by jeanPerier).
[flang] add attribute to trim runtime implementation establish call
Feb 24 2021, 9:55 AM
jeanPerier closed D97395: [flang] add attribute to trim runtime implementation establish call.
Feb 24 2021, 9:54 AM · Restricted Project, Restricted Project
jeanPerier requested review of D97395: [flang] add attribute to trim runtime implementation establish call.
Feb 24 2021, 8:47 AM · Restricted Project, Restricted Project
jeanPerier accepted D97331: [flang][fir] Add zero_bits operation..
Feb 24 2021, 12:44 AM · Restricted Project, Restricted Project

Feb 23 2021

jeanPerier abandoned D97189: [flang][NFC] Add source line to TODO message.

Duplicate of D97190.

Feb 23 2021, 12:07 AM · Restricted Project
jeanPerier committed rG36ea1cac1a42: [flang][NFC] Add source line to lowering TODO messages (authored by jeanPerier).
[flang][NFC] Add source line to lowering TODO messages
Feb 23 2021, 12:02 AM
jeanPerier closed D97190: [flang][NFC] Add source line to lowering TODO messages.
Feb 23 2021, 12:02 AM · Restricted Project, Restricted Project

Feb 22 2021

jeanPerier requested review of D97190: [flang][NFC] Add source line to lowering TODO messages.
Feb 22 2021, 5:36 AM · Restricted Project, Restricted Project
jeanPerier requested review of D97189: [flang][NFC] Add source line to TODO message.
Feb 22 2021, 5:17 AM · Restricted Project

Feb 19 2021

jeanPerier accepted D96987: [flang][fir][NFC] Move remaining types to TableGen type definition.

Thanks for the refactoring. Looks good to me.

Feb 19 2021, 9:36 AM · Restricted Project

Feb 11 2021

jeanPerier accepted D96362: [flang][fir] Update the kind mapping class..
Feb 11 2021, 9:17 AM · Restricted Project, Restricted Project

Feb 9 2021

jeanPerier accepted D96172: [flang][fir] Add shape, shape_shift, and slice types..
Feb 9 2021, 1:26 AM · Restricted Project, Restricted Project

Feb 7 2021

jeanPerier accepted D96183: [flang][fir] Update FIR's character type..
Feb 7 2021, 11:23 PM · Restricted Project

Feb 2 2021

jeanPerier accepted D95828: [flang][NFC] Update #include and comment..
Feb 2 2021, 3:45 AM · Restricted Project, Restricted Project

Feb 1 2021

jeanPerier accepted D95751: [flang][NFC] Rename complex type..
Feb 1 2021, 4:05 AM · Restricted Project, Restricted Project

Jan 22 2021

jeanPerier accepted D94716: [flang] Correct shape analysis for transformational intrinsic functions.

Looks good to me.

Jan 22 2021, 9:42 AM · Restricted Project, Restricted Project

Jan 8 2021

jeanPerier accepted D94017: [flang] Fix bogus message on internal subprogram with alternate return.

Looks good to me, thanks Pete.

Jan 8 2021, 12:36 AM · Restricted Project, Restricted Project

Jan 7 2021

jeanPerier requested changes to D94017: [flang] Fix bogus message on internal subprogram with alternate return.

@schweitz, as I understand things, the current lowering code gets the value of alternate return labels from the parse tree. Do you plan to change that once their available in the values of the ActualArgument?

That would make sens IMHO.
Otherwise, the change you have should work OK with lowering, except for the ProcedureRef hasAlternateReturns that is now wrong and would make lowering fail (see inlined comment).

Jan 7 2021, 4:42 AM · Restricted Project, Restricted Project

Dec 16 2020

jeanPerier accepted D93321: [flang] Fix crash in folding (#48437).

Looks good to me

Dec 16 2020, 1:03 AM · Restricted Project, Restricted Project

Dec 8 2020

jeanPerier accepted D92802: [flang] Implement derived type description table encoding.
Dec 8 2020, 9:26 AM · Restricted Project, Restricted Project

Nov 16 2020

jeanPerier accepted D91443: [flang] Unsplit COMPLEX operations.

Looks good to me.

Nov 16 2020, 3:33 AM · Restricted Project, Restricted Project

Nov 12 2020

jeanPerier accepted D91215: [flang] Implement runtime support for basic ALLOCATE/DEALLOCATE.

Thanks, looks good to me.

Nov 12 2020, 10:12 AM · Restricted Project, Restricted Project
jeanPerier added inline comments to D91215: [flang] Implement runtime support for basic ALLOCATE/DEALLOCATE.
Nov 12 2020, 6:54 AM · Restricted Project, Restricted Project

Nov 10 2020

jeanPerier accepted D91213: [flang] Add isnan() extension intrinsic function, with folding.

Looks good to me

Nov 10 2020, 11:35 PM · Restricted Project, Restricted Project

Nov 2 2020

jeanPerier accepted D90494: [flang] Allow array constructor implied DO loop indices as constant expressions.

Looks good to me

Nov 2 2020, 12:46 AM · Restricted Project, Restricted Project
jeanPerier accepted D90500: [flang] Design document for runtime derived type descriptions (NFC).

Looks good me

Nov 2 2020, 12:44 AM · Restricted Project, Restricted Project

Oct 26 2020

jeanPerier committed rG878b526409ac: [flang] Tighten rules to resolve procedure as intrinsic procedure (authored by jeanPerier).
[flang] Tighten rules to resolve procedure as intrinsic procedure
Oct 26 2020, 3:27 AM
jeanPerier closed D90049: [flang] Tighten rules to resolve procedure as intrinsic procedure.
Oct 26 2020, 3:27 AM · Restricted Project, Restricted Project
jeanPerier updated the diff for D90049: [flang] Tighten rules to resolve procedure as intrinsic procedure.

Thanks for the reviews. Fix typos in documentation an error message.

Oct 26 2020, 3:22 AM · Restricted Project, Restricted Project

Oct 23 2020

jeanPerier requested review of D90049: [flang] Tighten rules to resolve procedure as intrinsic procedure.
Oct 23 2020, 8:37 AM · Restricted Project, Restricted Project

Oct 20 2020

jeanPerier committed rG29d1a494477d: [flang] Document and use intrinsic subroutine argument intents (authored by jeanPerier).
[flang] Document and use intrinsic subroutine argument intents
Oct 20 2020, 5:11 AM
jeanPerier closed D89473: [flang] Document and use intrinsic subroutine argument intents.
Oct 20 2020, 5:11 AM · Restricted Project, Restricted Project

Oct 19 2020

jeanPerier updated the summary of D89473: [flang] Document and use intrinsic subroutine argument intents.
Oct 19 2020, 4:19 AM · Restricted Project, Restricted Project
jeanPerier added inline comments to D89473: [flang] Document and use intrinsic subroutine argument intents.
Oct 19 2020, 4:15 AM · Restricted Project, Restricted Project
jeanPerier updated the diff for D89473: [flang] Document and use intrinsic subroutine argument intents.

Only allow INTENT(OUT) allocatable coarrays in intrinsics where they are
explicitly allowed intstead of all intrinsics.

Oct 19 2020, 4:12 AM · Restricted Project, Restricted Project

Oct 16 2020

jeanPerier added inline comments to D89473: [flang] Document and use intrinsic subroutine argument intents.
Oct 16 2020, 1:21 AM · Restricted Project, Restricted Project
jeanPerier updated the diff for D89473: [flang] Document and use intrinsic subroutine argument intents.

Change DummyArgument::HasIntent to DummyArgument::GetIntent.

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