Page MenuHomePhabricator
People craig.rasmussen

craig.rasmussen (Craig E Rasmussen)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 20 2021, 12:59 PM (23 w, 6 h)

Recent Activity
View All

Today

craig.rasmussen added a comment to D110356: [flang] Make builtin types more easily accessible; use them.
In D110356#3026290, @klausler wrote:
In D110356#3026230, @craig.rasmussen wrote:
In D110356#3021316, @klausler wrote:

Craig R.: any comments?

Oh, you actually want me to comment on the change? I think you probably want me to use it. I need to take some time to digest the changes.

Would you like me to wait for your review comments before landing this patch?

Tue, Sep 28, 5:13 PM

Yesterday

craig.rasmussen added a comment to D100897: [flang] Make 'team_number()' an intrinsic function.

This need to be updated for changes in https://reviews.llvm.org/D110356

Mon, Sep 27, 4:48 PM · Restricted Project, Restricted Project
craig.rasmussen added a comment to D110356: [flang] Make builtin types more easily accessible; use them.
In D110356#3021316, @klausler wrote:

Craig R.: any comments?

Mon, Sep 27, 4:41 PM
craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Mon, Sep 27, 4:18 PM · Restricted Project, Restricted Project
craig.rasmussen updated the diff for D100897: [flang] Make 'team_number()' an intrinsic function.

Inlined implementation of call to semantics::IsTeamType. This is necessary because the semantics::IsTeamType symbol in undefined at the time libFortranEvalute is built.

Mon, Sep 27, 4:11 PM · Restricted Project, Restricted Project
craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Mon, Sep 27, 2:06 PM · Restricted Project, Restricted Project

Tue, Sep 21

craig.rasmussen added a reviewer for D100897: [flang] Make 'team_number()' an intrinsic function: jeanPerier.
Tue, Sep 21, 3:16 PM · Restricted Project, Restricted Project

Mon, Sep 20

craig.rasmussen updated the summary of D100897: [flang] Make 'team_number()' an intrinsic function.
Mon, Sep 20, 1:34 PM · Restricted Project, Restricted Project
craig.rasmussen updated the diff for D100897: [flang] Make 'team_number()' an intrinsic function.

Implemented TEAM_NUMBER intrinsic function without use of MissingDerivedType() hack.

Mon, Sep 20, 1:28 PM · Restricted Project, Restricted Project

Thu, Sep 16

craig.rasmussen committed rGed921282e551: [flang] Make 'this_image()' an intrinsic function (authored by craig.rasmussen).
[flang] Make 'this_image()' an intrinsic function
Thu, Sep 16, 10:19 PM
craig.rasmussen added a comment to D109547: [flang] Add runtime interface for NUM_IMAGES.

Hopefully this will open the issue and is the correct way to correct a push that was reverted.

Thu, Sep 16, 3:13 PM · Restricted Project
craig.rasmussen updated the diff for D109547: [flang] Add runtime interface for NUM_IMAGES.

Inlined call to Fortran::semantics::IsCoarray (one line change) to fix link error.

Thu, Sep 16, 3:06 PM · Restricted Project

Mon, Sep 13

craig.rasmussen updated subscribers of D109547: [flang] Add runtime interface for NUM_IMAGES.

Peter Klausler wrote:

Or you could have two entry points, one with a team number and one without, and avoid exposing this kind of trick in the API. It could still be used as a hidden part of a common implementation.

Mon, Sep 13, 4:08 PM · Restricted Project
craig.rasmussen updated the diff for D109547: [flang] Add runtime interface for NUM_IMAGES.

Added comments explaining why team_number=0 is a good default value when NUM_IMAGES is called without an actual argument for TEAM_NUMBER.

Mon, Sep 13, 12:34 PM · Restricted Project
craig.rasmussen updated the diff for D109547: [flang] Add runtime interface for NUM_IMAGES.

Changed location of header file to flang/include/flang/Runtime/coarray.h.

Mon, Sep 13, 12:14 PM · Restricted Project

Thu, Sep 9

craig.rasmussen added inline comments to D109547: [flang] Add runtime interface for NUM_IMAGES.
Thu, Sep 9, 3:14 PM · Restricted Project
craig.rasmussen requested review of D109547: [flang] Add runtime interface for NUM_IMAGES.
Thu, Sep 9, 2:52 PM · Restricted Project
craig.rasmussen added a comment to D100897: [flang] Make 'team_number()' an intrinsic function.

In a response @klausler asked:

Thu, Sep 9, 2:45 PM · Restricted Project, Restricted Project
craig.rasmussen added a reviewer for D100897: [flang] Make 'team_number()' an intrinsic function: PeteSteinfeld.
Thu, Sep 9, 2:29 PM · Restricted Project, Restricted Project
craig.rasmussen added a reviewer for D108059: [flang] Make 'this_image()' an intrinsic function: PeteSteinfeld.
Thu, Sep 9, 2:28 PM · Restricted Project, Restricted Project

Fri, Sep 3

craig.rasmussen added a comment to D100897: [flang] Make 'team_number()' an intrinsic function.

@klausler, are any further changes required to get this approved?

Fri, Sep 3, 11:30 AM · Restricted Project, Restricted Project

Aug 18 2021

craig.rasmussen added a comment to D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

ping

Aug 18 2021, 11:28 AM · Restricted Project

Aug 13 2021

craig.rasmussen requested review of D108059: [flang] Make 'this_image()' an intrinsic function.
Aug 13 2021, 4:00 PM · Restricted Project, Restricted Project

Aug 2 2021

craig.rasmussen updated the diff for D100897: [flang] Make 'team_number()' an intrinsic function.
  1. Added "! REQUIRES: shell" to test file so as not to trip on windows
  2. Ran git clang-format
Aug 2 2021, 2:41 PM · Restricted Project, Restricted Project

Jul 26 2021

craig.rasmussen added a comment to D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

How do I ping? I have another pull request that is also in limbo.

Jul 26 2021, 12:12 PM · Restricted Project

Jul 21 2021

craig.rasmussen added a comment to D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

Requesting review (is there a better way to request a review)

Jul 21 2021, 4:06 PM · Restricted Project
craig.rasmussen updated the diff for D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

Made simplification suggested by reviewer:

Jul 21 2021, 11:09 AM · Restricted Project

Jul 8 2021

craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Jul 8 2021, 12:07 PM · Restricted Project, Restricted Project
craig.rasmussen added a comment to D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

Fixed patch and added reviewer.

Jul 8 2021, 11:51 AM · Restricted Project
craig.rasmussen added reviewers for D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0: jhenderson, sscalpone.
Jul 8 2021, 11:50 AM · Restricted Project
craig.rasmussen updated the diff for D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.

Forgot to commit changes so patch/diff was not correct.

Jul 8 2021, 11:47 AM · Restricted Project

Jul 7 2021

craig.rasmussen updated the summary of D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.
Jul 7 2021, 2:57 PM · Restricted Project
craig.rasmussen requested review of D105595: [flang] Compile error instantiating std::clock_t on Apple clang version 11.0.0.
Jul 7 2021, 2:51 PM · Restricted Project
craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Jul 7 2021, 1:26 PM · Restricted Project, Restricted Project
craig.rasmussen added a comment to D100897: [flang] Make 'team_number()' an intrinsic function.

Added line comments.

Jul 7 2021, 12:46 PM · Restricted Project, Restricted Project
craig.rasmussen updated the diff for D100897: [flang] Make 'team_number()' an intrinsic function.

Added the static function MissingDerivedType() to allow the creation of a DynamicType with TypeCategory::Derived when a semantics::DerivedTypeSpec is not directly available. This allows a missing actual argument for an optional dummy.

Jul 7 2021, 12:23 PM · Restricted Project, Restricted Project

Apr 28 2021

craig.rasmussen added a comment to D100897: [flang] Make 'team_number()' an intrinsic function.

I forgot to add that there is one test failing at the moment which makes no sense to me.

Apr 28 2021, 10:31 AM · Restricted Project, Restricted Project
craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Apr 28 2021, 10:11 AM · Restricted Project, Restricted Project
craig.rasmussen updated the diff for D100897: [flang] Make 'team_number()' an intrinsic function.
  1. Fixed team argument in call to team_number so that it is optional.
  2. Fixed teamType kind in flang/lib/Evaluate/intrinsics.cpp
  3. Added team_number test to flang/unittests/Evaluate/intrinsics.cpp
Apr 28 2021, 10:06 AM · Restricted Project, Restricted Project

Apr 20 2021

craig.rasmussen added inline comments to D100897: [flang] Make 'team_number()' an intrinsic function.
Apr 20 2021, 3:49 PM · Restricted Project, Restricted Project
craig.rasmussen requested review of D100897: [flang] Make 'team_number()' an intrinsic function.
Apr 20 2021, 3:27 PM · Restricted Project, Restricted Project