Page MenuHomePhabricator
Feed Advanced Search

Fri, Jun 18

ashermancinelli committed rG5c189d30e60a: [OpenMP] Update FAQ for enabling cuda offloading (authored by ashermancinelli).
[OpenMP] Update FAQ for enabling cuda offloading
Fri, Jun 18, 10:57 AM
ashermancinelli closed D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.
Fri, Jun 18, 10:57 AM · Restricted Project
ashermancinelli updated the diff for D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.

Update language to use cmake variables over ugly environment variable workaround.

Fri, Jun 18, 10:50 AM · Restricted Project

Thu, Jun 17

ashermancinelli updated the diff for D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.

Document use of CMake variable GCC_INSTALL_PREFIX over ugly workaround.

Thu, Jun 17, 3:58 PM · Restricted Project
ashermancinelli added a comment to D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.

I've hopefully addressed your comments. Thanks for the review @jhuber6!

Thu, Jun 17, 11:36 AM · Restricted Project
ashermancinelli updated the diff for D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.

Remove CMake example. Use more generic language.

Thu, Jun 17, 11:35 AM · Restricted Project
ashermancinelli requested review of D104474: [OpenMP] Update FAQ with additional instructions for enabling cuda offloading.
Thu, Jun 17, 10:20 AM · Restricted Project

Tue, Jun 15

ashermancinelli accepted D104320: [flang] Move `external-hello-world` to flang/examples.
Tue, Jun 15, 1:40 PM · Restricted Project, Restricted Project

Mon, Jun 14

ashermancinelli committed rGc58cf692f419: [flang] Move buffer runtime test to GTest (authored by ashermancinelli).
[flang] Move buffer runtime test to GTest
Mon, Jun 14, 10:16 AM
ashermancinelli closed D102335: [flang] Move buffer runtime test to GTest.
Mon, Jun 14, 10:16 AM · Restricted Project, Restricted Project

Fri, Jun 11

ashermancinelli updated the diff for D102335: [flang] Move buffer runtime test to GTest.

Add LLVM header comment to buffer test.

Fri, Jun 11, 8:03 AM · Restricted Project, Restricted Project

Thu, Jun 10

ashermancinelli added reviewers for D102335: [flang] Move buffer runtime test to GTest: klausler, tskeith.
Thu, Jun 10, 10:35 AM · Restricted Project, Restricted Project

Wed, Jun 9

ashermancinelli added a comment to D102335: [flang] Move buffer runtime test to GTest.

@awarzynski Sounds good! I'll add a few more reviewers and wait a bit before merging.

Wed, Jun 9, 9:49 AM · Restricted Project, Restricted Project

Thu, Jun 3

ashermancinelli accepted D103554: [flang][driver] Add checks for missing option arguments.

lgtm!

Thu, Jun 3, 8:44 PM · Restricted Project, Restricted Project

May 17 2021

ashermancinelli updated the diff for D101261: [flang] Lift -Werror checks into local functions.

Update revision with supporting functionality test.

May 17 2021, 4:49 PM · Restricted Project, Restricted Project

May 12 2021

ashermancinelli added a comment to D102335: [flang] Move buffer runtime test to GTest.

@awarzynski This is just the minimum to pull the test into gtest. I would like to add documentation and death tests for failure conditions - do you have any suggestions for this? I think once we have those two criteria met we can add more reviewers.

May 12 2021, 10:12 AM · Restricted Project, Restricted Project
ashermancinelli requested review of D102335: [flang] Move buffer runtime test to GTest.
May 12 2021, 10:11 AM · Restricted Project, Restricted Project

Apr 27 2021

ashermancinelli committed rG4abba775a39a: [flang] Add format test to GTest suite (authored by ashermancinelli).
[flang] Add format test to GTest suite
Apr 27 2021, 7:46 AM
ashermancinelli closed D100765: [flang] Update format tests to use GTest.
Apr 27 2021, 7:46 AM · Restricted Project, Restricted Project

Apr 25 2021

ashermancinelli added a comment to D101261: [flang] Lift -Werror checks into local functions.

Notice that in PrescanAndSemaAction, the first check fails with a scanning error, and the second with a parsing error. This is not the case for PrescanAndParseAction, which makes me think the second check should fail with a parsing error, and the tests should be adjusted in the same way.

Apr 25 2021, 10:32 AM · Restricted Project, Restricted Project
ashermancinelli requested review of D101261: [flang] Lift -Werror checks into local functions.
Apr 25 2021, 10:30 AM · Restricted Project, Restricted Project

Apr 23 2021

ashermancinelli added a comment to D100765: [flang] Update format tests to use GTest.

Thanks again for the rigorous review!

Apr 23 2021, 7:37 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D100765: [flang] Update format tests to use GTest.

Pull all of previous test harness into current implementation.

Apr 23 2021, 7:36 AM · Restricted Project, Restricted Project
ashermancinelli accepted D100649: [flang][driver] Fine-tune `-fdebug-dump-symbols`.

LGTM!

Apr 23 2021, 7:03 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D100765: [flang] Update format tests to use GTest.

Resolve warnings against const auto without &.

Apr 23 2021, 6:57 AM · Restricted Project, Restricted Project

Apr 22 2021

ashermancinelli updated the diff for D100765: [flang] Update format tests to use GTest.

Remove erroneous static qualifiers. Register all tests with crash handler.

Apr 22 2021, 11:04 AM · Restricted Project, Restricted Project

Apr 20 2021

ashermancinelli added a comment to D100442: [flang][msvc] Fix compilation of RuntimeGtest tests..

Merged in commit 080d48f279e2.

Apr 20 2021, 2:45 PM · Restricted Project, Restricted Project
ashermancinelli committed rG080d48f279e2: [flang][msvc] Fix compilation of RuntimeGtest (authored by Mehdi Chinoune <MehdiChinoune>).
[flang][msvc] Fix compilation of RuntimeGtest
Apr 20 2021, 2:37 PM
ashermancinelli closed D100442: [flang][msvc] Fix compilation of RuntimeGtest tests..
Apr 20 2021, 2:37 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D100442: [flang][msvc] Fix compilation of RuntimeGtest tests..

Could someone push this change p, please.

Apr 20 2021, 11:56 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D100765: [flang] Update format tests to use GTest.

Resolve warnings by removing utility function.

Apr 20 2021, 11:46 AM · Restricted Project, Restricted Project

Apr 19 2021

ashermancinelli requested review of D100765: [flang] Update format tests to use GTest.
Apr 19 2021, 7:41 AM · Restricted Project, Restricted Project
ashermancinelli closed D99614: [flang] Update list-input test to use GTest.

Merged in ccef0adc5968.

Apr 19 2021, 7:39 AM · Restricted Project, Restricted Project

Apr 15 2021

ashermancinelli accepted D100442: [flang][msvc] Fix compilation of RuntimeGtest tests..
Apr 15 2021, 1:33 PM · Restricted Project, Restricted Project
ashermancinelli committed rGccef0adc5968: [flang] Add list input test to GTest suite (authored by ashermancinelli).
[flang] Add list input test to GTest suite
Apr 15 2021, 9:34 AM

Apr 14 2021

ashermancinelli added inline comments to D100442: [flang][msvc] Fix compilation of RuntimeGtest tests..
Apr 14 2021, 7:57 AM · Restricted Project, Restricted Project

Apr 12 2021

ashermancinelli updated the diff for D99614: [flang] Update list-input test to use GTest.

Use more descriptive variable names.

Apr 12 2021, 8:28 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D99614: [flang] Update list-input test to use GTest.

@awarzynski Thanks again for the review! I have another patch ready to address this.

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

Apr 9 2021

ashermancinelli added a comment to D99614: [flang] Update list-input test to use GTest.

@awarzynski Thank you for the fantastic review comments! I've tried to address most of them - please let me know if this looks good to you.

Apr 9 2021, 4:54 PM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D99614: [flang] Update list-input test to use GTest.

Use more descriptive variable names and better comment tests. Address clang-tidy warnings and phabricator reviews.

Apr 9 2021, 4:52 PM · Restricted Project, Restricted Project

Apr 6 2021

ashermancinelli accepted D99645: [flang][driver] Add debug options not requiring semantic checks.
Apr 6 2021, 10:46 AM · Restricted Project, Restricted Project
ashermancinelli accepted D99353: [driver] Make `clang` warn rather then error on `flang` options.
Apr 6 2021, 10:22 AM · Restricted Project, Restricted Project
ashermancinelli added inline comments to D99645: [flang][driver] Add debug options not requiring semantic checks.
Apr 6 2021, 10:20 AM · Restricted Project, Restricted Project

Apr 5 2021

ashermancinelli updated the diff for D99614: [flang] Update list-input test to use GTest.

Break original test into two test cases. Add simple sanity-check test cases. Add death test to verify failure message.

Apr 5 2021, 2:24 PM · Restricted Project, Restricted Project

Mar 30 2021

ashermancinelli added a reviewer for D99614: [flang] Update list-input test to use GTest: klausler.
Mar 30 2021, 12:20 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D99614: [flang] Update list-input test to use GTest.

If deemed reasonable, I'm happy to extend this to test more of the api as well - I'm open to suggestions. I'm also happy to keep this patch very short.

Mar 30 2021, 11:38 AM · Restricted Project, Restricted Project
ashermancinelli requested review of D99614: [flang] Update list-input test to use GTest.
Mar 30 2021, 11:36 AM · Restricted Project, Restricted Project

Mar 29 2021

ashermancinelli updated the diff for D98303: [flang] Update output format test to use GTest.

Use GTest EXPECT_EQ over potentially ambiguous return value, redirect SUT IO to stderr.

Mar 29 2021, 8:53 AM · Restricted Project, Restricted Project

Mar 19 2021

ashermancinelli updated the diff for D98303: [flang] Update output format test to use GTest.

Fix header include order.

Mar 19 2021, 12:09 PM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D98303: [flang] Update output format test to use GTest.

Fix clang-format and clang-tidy warnings.

Mar 19 2021, 10:38 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D98303: [flang] Update output format test to use GTest.

@klausler D98652 hopefully demonstrates that runtime crashes are caught by death tests. This revision has been rebased on D98652 - I hope this sufficiently demonstrates that these tests accurately catch crashes. We could also work on another patch that intentionally crashes each of the APIs tested above not already tested in D98652.

I wasn't asking about crashes in the runtime. I was asking about catching incorrect results and reporting failures.

I will try again. If you modify the decimal or runtime library in such a way as to cause f18 to emit incorrectly formatted results, does your rewritten test fail? Let me know when you have confirmed this by observation.

Mar 19 2021, 10:35 AM · Restricted Project, Restricted Project

Mar 18 2021

ashermancinelli added a comment to D98303: [flang] Update output format test to use GTest.

@klausler D98652 hopefully demonstrates that runtime crashes are caught by death tests. This revision has been rebased on D98652 - I hope this sufficiently demonstrates that these tests accurately catch crashes. We could also work on another patch that intentionally crashes each of the APIs tested above not already tested in D98652.

Mar 18 2021, 4:08 PM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D98303: [flang] Update output format test to use GTest.

Rebase on recent main branch to incorporate terminator crash tests

Mar 18 2021, 4:05 PM · Restricted Project, Restricted Project

Mar 17 2021

ashermancinelli added inline comments to D98652: [flang] Unittests for runtime terminator.
Mar 17 2021, 10:33 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D98652: [flang] Unittests for runtime terminator.

Remove superfluous comments

Mar 17 2021, 10:30 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D98652: [flang] Unittests for runtime terminator.

Fix header guards to conform to LLVM style guide

Mar 17 2021, 10:26 AM · Restricted Project, Restricted Project

Mar 16 2021

ashermancinelli updated the diff for D98652: [flang] Unittests for runtime terminator.

More thoroughly comment runtime crash tests.

Mar 16 2021, 1:30 PM · Restricted Project, Restricted Project

Mar 15 2021

ashermancinelli added a comment to D98601: [flang][unittest] Add unit tests for runtime's crash handler.

I have submitted D98652 to discuss similar efforts.

Mar 15 2021, 12:49 PM · Restricted Project, Restricted Project
ashermancinelli requested review of D98652: [flang] Unittests for runtime terminator.
Mar 15 2021, 12:48 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D98601: [flang][unittest] Add unit tests for runtime's crash handler.

@awarzynski I've actually been working on a similar patch, perhaps we could merge our efforts?

Mar 15 2021, 11:30 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D98601: [flang][unittest] Add unit tests for runtime's crash handler.

These tests seem to adequately verify the runtime crash mechanism and would support other gtest tests that must catch runtime crashes.

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

Mar 14 2021

ashermancinelli added a comment to D98553: [flang] Runtime: implement INDEX intrinsic function.

LGTM

Mar 14 2021, 2:43 PM · Restricted Project, Restricted Project

Mar 10 2021

ashermancinelli added a comment to D98303: [flang] Update output format test to use GTest.

@klausler I think I better understand your concerns now. How would you feel about this patch if I un-removed the original test, we refine this test, and ten

Mar 10 2021, 2:37 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D98303: [flang] Update output format test to use GTest.

@klausler I reproduce all of the same test cases, so the test coverage should be identical and should catch all of the same bugs. I'd like to increase coverage, but the patch as-is should keep it the same.

Mar 10 2021, 10:04 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D98303: [flang] Update output format test to use GTest.

Use fully qualified auto to address clang-tidy warnings.

Mar 10 2021, 8:23 AM · Restricted Project, Restricted Project

Mar 9 2021

ashermancinelli added a comment to D98303: [flang] Update output format test to use GTest.

Has this version of the tests been validated for its ability to still find bugs in a broken runtime or decimal conversion library?

Mar 9 2021, 4:36 PM · Restricted Project, Restricted Project
ashermancinelli requested review of D98303: [flang] Update output format test to use GTest.
Mar 9 2021, 3:49 PM · Restricted Project, Restricted Project

Mar 8 2021

ashermancinelli updated the summary of D97403: [flang] update character tests to use gtest.
Mar 8 2021, 11:37 AM · Restricted Project, Restricted Project
ashermancinelli updated the summary of D97403: [flang] update character tests to use gtest.
Mar 8 2021, 11:32 AM · Restricted Project, Restricted Project
ashermancinelli retitled D97403: [flang] update character tests to use gtest from [flang] updated character tests to gtest to [flang] update character tests to use gtest.
Mar 8 2021, 11:26 AM · Restricted Project, Restricted Project

Mar 7 2021

ashermancinelli updated the diff for D97403: [flang] update character tests to use gtest.

Updated naming conventions and documentation thanks to comments and suggestions from @awarzynski and @tskeith.

Mar 7 2021, 3:55 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

I've updated the revision - thanks for your comments @awarzynski and @tskeith! Please let me know what you think of the naming conventions I went with in this revision.

Mar 7 2021, 3:54 PM · Restricted Project, Restricted Project
ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

ALL_CAPS is usually reserved for macros...
but we should follow the existing precedent in the project...

Mar 7 2021, 12:18 PM · Restricted Project, Restricted Project

Mar 4 2021

ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Mar 4 2021, 11:33 AM · Restricted Project, Restricted Project
ashermancinelli updated the diff for D97403: [flang] update character tests to use gtest.

I've added tests for scan and verify functions, and cleaned up the comparison tests. I think the std::get method is a bit cleaner than what we had before, but I would love to hear other opinions on this.

Mar 4 2021, 11:26 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

@awarzynski Sure! I'll add corresponding tests and have another patch up soon.

Mar 4 2021, 9:50 AM · Restricted Project, Restricted Project

Mar 3 2021

ashermancinelli added a comment to D97457: [flang][driver] Add `-fdebug-dump-parsing-log`.

It is unintuitive to me that we are mixing feature and action flags, however I don't have a better suggestion. LGTM.

Mar 3 2021, 3:13 PM · Restricted Project, Restricted Project

Mar 1 2021

ashermancinelli updated the diff for D97403: [flang] update character tests to use gtest.

Updated documentation, added a new test case based on conversation in the previous diff, and removed old character tests.

Mar 1 2021, 1:28 PM · Restricted Project, Restricted Project
ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Mar 1 2021, 1:12 PM · Restricted Project, Restricted Project

Feb 28 2021

ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Feb 28 2021, 11:38 AM · Restricted Project, Restricted Project

Feb 26 2021

ashermancinelli updated the diff for D97403: [flang] update character tests to use gtest.
  • Templated character tests over all char types
  • Removed unneeded fixture
  • Documentation and better naming
Feb 26 2021, 8:46 AM · Restricted Project, Restricted Project

Feb 25 2021

ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

I am also thinking that since procedures from character.cpp don't crash (at least the ones that you test), we don't really need CatchCrash just yet (though it would be great to see it being tested). Perhaps the test fixture could be simplified?

Feb 25 2021, 5:03 PM · Restricted Project, Restricted Project
ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Feb 25 2021, 2:58 PM · Restricted Project, Restricted Project
ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Feb 25 2021, 11:36 AM · Restricted Project, Restricted Project
ashermancinelli added inline comments to D97403: [flang] update character tests to use gtest.
Feb 25 2021, 11:06 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

Thank you for the review! It appears that many of your comments are due to lack of attention to detail when moving the test bodies over from the old runtime tests directory to this one (eg your comments on lines 28 and 52) - I'll revisit all of your comments and see where I can improve upon the old tests as well.

Feb 25 2021, 10:55 AM · Restricted Project, Restricted Project

Feb 24 2021

ashermancinelli added a comment to D97403: [flang] update character tests to use gtest.

I no longer modify anything in the old runtime tests directory, so this patch should be much more straightforward.

Feb 24 2021, 11:10 AM · Restricted Project, Restricted Project
ashermancinelli requested review of D97403: [flang] update character tests to use gtest.
Feb 24 2021, 11:09 AM · Restricted Project, Restricted Project
ashermancinelli abandoned D97381: [flang] Change existing runtime tests to use gtest.

@awarzynski Looks like I mistakenly abandoned the last revision for this. Should I keep this one open or should we stick to the abandoned one since you've already added other reviewers to it?

Feb 24 2021, 7:34 AM · Restricted Project
ashermancinelli added a comment to D97349: [flang] Change existing runtime tests to use gtest.
Feb 24 2021, 7:24 AM · Restricted Project, Restricted Project
ashermancinelli abandoned D97349: [flang] Change existing runtime tests to use gtest.

Thanks for the patch! Could you please clang-format the patch(this is will remove lot of Lint warning/noise) and ease up the review.
If you're using arcanist then that will automatically take care of formatting(provided clang-format should be in system path.) Or if you're using web UI then you may have to do it manually (git-clang-format HEAD~).
More details: https://llvm.org/docs/Phabricator.html

Feb 24 2021, 7:22 AM · Restricted Project, Restricted Project
ashermancinelli added a comment to D97381: [flang] Change existing runtime tests to use gtest.

@awarzynski Looks like I mistakenly abandoned the last revision for this. Should I keep this one open or should we stick to the abandoned one since you've already added other reviewers to it?

Feb 24 2021, 7:17 AM · Restricted Project
ashermancinelli requested review of D97381: [flang] Change existing runtime tests to use gtest.
Feb 24 2021, 7:01 AM · Restricted Project

Feb 23 2021

ashermancinelli requested review of D97349: [flang] Change existing runtime tests to use gtest.
Feb 23 2021, 5:12 PM · Restricted Project, Restricted Project