Page MenuHomePhabricator

tlwilmar (Terry Wilmarth)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 20 2015, 1:32 PM (311 w, 5 d)

Recent Activity

Mar 5 2021

tlwilmar closed D97019: [OpenMP] libomp: Add masked construct, change proc-bind keyword, deprecate master construct.
Mar 5 2021, 12:47 PM · Restricted Project
tlwilmar added a comment to D97019: [OpenMP] libomp: Add masked construct, change proc-bind keyword, deprecate master construct.

Hi Johnny,
When I just commit via the instructions on our wiki, I just see this separate e-mail labeled "[Diffusion]" which looks like a separate review in Phabricator. When others push their changes, it appears attached to the original phabricator review, and seems to close it automagically. What am I missing there?

Mar 5 2021, 7:37 AM · Restricted Project
tlwilmar committed rG97d000cfc6d4: Added API for "masked" construct via two entrypoints: __kmpc_masked, (authored by tlwilmar).
Added API for "masked" construct via two entrypoints: __kmpc_masked,
Mar 5 2021, 7:30 AM

Feb 26 2021

tlwilmar updated the diff for D97019: [OpenMP] libomp: Add masked construct, change proc-bind keyword, deprecate master construct.

Added masked to cons_text_c.

Feb 26 2021, 5:34 PM · Restricted Project

Feb 18 2021

tlwilmar requested review of D97019: [OpenMP] libomp: Add masked construct, change proc-bind keyword, deprecate master construct.
Feb 18 2021, 7:28 PM · Restricted Project

Jan 29 2021

tlwilmar committed rG67773681c0ef: [OpenMP] Add environment variable to force monotonic dynamic scheduling (authored by jlpeyton).
[OpenMP] Add environment variable to force monotonic dynamic scheduling
Jan 29 2021, 10:27 AM
tlwilmar closed D95263: [OpenMP] Add environment variable to force monotonic dynamic scheduling.
Jan 29 2021, 10:26 AM · Restricted Project

Jan 28 2021

tlwilmar updated the diff for D95263: [OpenMP] Add environment variable to force monotonic dynamic scheduling.

Addressed Nawrin's comment fixes.

Jan 28 2021, 9:36 AM · Restricted Project

Jan 27 2021

tlwilmar accepted D95157: [OpenMP] Add CPUID leaf 1f topology discovery.

LGTM

Jan 27 2021, 7:23 AM · Restricted Project

Jan 26 2021

tlwilmar accepted D95156: [OpenMP] Update HWLOC topology detection for 2.0.x.

LGTM

Jan 26 2021, 7:24 AM · Restricted Project
tlwilmar accepted D95170: [OpenMP] Fix misleading warning for OMP_PLACES.

LGTM

Jan 26 2021, 7:02 AM · Restricted Project

Jan 22 2021

tlwilmar requested review of D95263: [OpenMP] Add environment variable to force monotonic dynamic scheduling.
Jan 22 2021, 12:58 PM · Restricted Project

Jan 7 2021

tlwilmar requested review of D94241: [OpenMP] Fix hierarchical barrier.
Jan 7 2021, 9:20 AM · Restricted Project

Dec 17 2020

tlwilmar added a reviewer for D92942: Handle implicit conversion warnings: jlpeyton.
Dec 17 2020, 8:50 AM · Restricted Project

Dec 9 2020

tlwilmar requested review of D92942: Handle implicit conversion warnings.
Dec 9 2020, 8:59 AM · Restricted Project

Dec 1 2020

tlwilmar requested review of D91189: Add support for Intel's umonitor/umwait.
Dec 1 2020, 9:35 AM · Restricted Project
tlwilmar updated the diff for D91189: Add support for Intel's umonitor/umwait.

One missed change added.

Dec 1 2020, 9:33 AM · Restricted Project

Nov 30 2020

tlwilmar reopened D91189: Add support for Intel's umonitor/umwait.

Build problems associated with target attribute for waitpkg should be fixed now. Please try the new diff.

Nov 30 2020, 9:37 PM · Restricted Project
tlwilmar updated the diff for D91189: Add support for Intel's umonitor/umwait.

Fixed build problems associated with target attribute for waitpkg.

Nov 30 2020, 9:36 PM · Restricted Project
tlwilmar updated the summary of D91189: Add support for Intel's umonitor/umwait.
Nov 30 2020, 9:32 PM · Restricted Project

Nov 16 2020

tlwilmar accepted D90867: [OpenMP][stats] reset serial state when re-entering serial region.

LGTM

Nov 16 2020, 12:26 PM · Restricted Project

Nov 10 2020

tlwilmar requested review of D91189: Add support for Intel's umonitor/umwait.
Nov 10 2020, 10:27 AM · Restricted Project

Oct 23 2019

tlwilmar accepted D68036: Tasks dependencies hash re-sizing fixed.

LGTM

Oct 23 2019, 11:00 AM · Restricted Project

Aug 5 2019

tlwilmar added a reviewer for D65714: Add support for libgomp's GOMP_*_nonmonotonic_* functions: jlpeyton.
Aug 5 2019, 7:51 AM · Restricted Project, Restricted Project

Jul 22 2019

tlwilmar accepted rL366716: Delete empty file.

LGTM

Jul 22 2019, 11:23 AM

Jul 12 2019

tlwilmar added inline comments to D64534: Remove OMP spec versioning.
Jul 12 2019, 1:10 PM · Restricted Project, Restricted Project
tlwilmar added a comment to D64534: Remove OMP spec versioning.

@tlwilmar D64625 conflicts with moving the headers. Let me know if you want to land this first, but I'd like to get the fix into the next release which is set to branch next week.

Jul 12 2019, 5:29 AM · Restricted Project, Restricted Project

Jul 10 2019

tlwilmar created D64534: Remove OMP spec versioning.
Jul 10 2019, 2:47 PM · Restricted Project, Restricted Project

Jun 27 2019

tlwilmar created D63891: Cleanup of unused code.
Jun 27 2019, 1:05 PM · Restricted Project, Restricted Project

Jun 18 2019

tlwilmar accepted D62485: New implementation of OpenMP 5.0 detached tasks..

LGTM

Jun 18 2019, 1:21 PM · Restricted Project, Restricted Project

Jun 17 2019

tlwilmar added a reviewer for D63009: [OpenMP] Add target task alloc function with device ID: hbae.
Jun 17 2019, 11:48 AM · Restricted Project, Restricted Project, Restricted Project

May 15 2019

tlwilmar accepted D61944: Fixed https://bugs.llvm.org/show_bug.cgi?id=41584.

LGTM!

May 15 2019, 7:21 AM · Restricted Project

Apr 23 2019

tlwilmar accepted D60983: [OpenMP] Eliminate some compiler warnings.

LGTM

Apr 23 2019, 12:18 PM · Restricted Project, Restricted Project

Apr 8 2019

tlwilmar accepted D59783: [OpenMP] Implement 5.0 memory management.

LGTM

Apr 8 2019, 10:35 AM · Restricted Project
tlwilmar added a comment to D59508: Clean up load balancing dynamic mode.

*ping*

Apr 8 2019, 10:24 AM · Restricted Project, Restricted Project

Apr 2 2019

tlwilmar accepted D59801: [OpenMP][Stats] Fix stats gathering for distribute and team clause.

LGTM

Apr 2 2019, 1:01 PM · Restricted Project, Restricted Project

Mar 18 2019

tlwilmar updated the summary of D59508: Clean up load balancing dynamic mode.
Mar 18 2019, 1:24 PM · Restricted Project, Restricted Project
tlwilmar created D59508: Clean up load balancing dynamic mode.
Mar 18 2019, 1:17 PM · Restricted Project, Restricted Project

Mar 15 2019

tlwilmar created D59428: Fix pause check with version info.
Mar 15 2019, 12:38 PM · Restricted Project, Restricted Project
tlwilmar added a comment to D58989: Remove deprecated taskq.

*ping*

Mar 15 2019, 10:58 AM · Restricted Project, Restricted Project

Mar 5 2019

tlwilmar created D58989: Remove deprecated taskq.
Mar 5 2019, 12:12 PM · Restricted Project, Restricted Project

Feb 27 2019

tlwilmar added a comment to D58148: Make use of sched_yield optional in runtime.

*ping*

Feb 27 2019, 2:05 PM · Restricted Project
tlwilmar added a comment to D58408: [5.0] Deprecate nest-var and associated features.

*ping*

Feb 27 2019, 2:05 PM · Restricted Project, Restricted Project

Feb 26 2019

tlwilmar added a reviewer for D58148: Make use of sched_yield optional in runtime: hbae.
Feb 26 2019, 1:23 PM · Restricted Project

Feb 19 2019

tlwilmar created D58408: [5.0] Deprecate nest-var and associated features.
Feb 19 2019, 2:44 PM · Restricted Project, Restricted Project

Feb 13 2019

tlwilmar accepted D57969: [OpenMP] Adding GOMP compatible cancellation.

LGTM

Feb 13 2019, 2:59 PM · Restricted Project, Restricted Project
tlwilmar created D58211: [5.0] Add omp_get_supported_active_levels().
Feb 13 2019, 2:57 PM · Restricted Project, Restricted Project

Feb 12 2019

tlwilmar created D58148: Make use of sched_yield optional in runtime.
Feb 12 2019, 2:12 PM · Restricted Project

Feb 11 2019

tlwilmar updated the diff for D56804: Fix thread_limits to work properly for teams construct.

Fix update for non-CG_root threads in __kmp_free_thread.

Feb 11 2019, 10:44 AM · Restricted Project, Restricted Project

Feb 8 2019

tlwilmar added a comment to D56804: Fix thread_limits to work properly for teams construct.

I was going for the per-data-environment specification for the ICV, though I agree it is only needed per contention group. Is there any consensus on this? Leave as is, or remove per-data-environment ICV?

Feb 8 2019, 9:56 AM · Restricted Project, Restricted Project

Jan 16 2019

tlwilmar created D56804: Fix thread_limits to work properly for teams construct.
Jan 16 2019, 1:12 PM · Restricted Project, Restricted Project
tlwilmar updated the diff for D55078: Add omp_pause_resource* API.

Sorry Andrey -- I missed the Windows fix. Added now.

Jan 16 2019, 11:58 AM · Restricted Project

Jan 15 2019

tlwilmar updated the diff for D55078: Add omp_pause_resource* API.

Renamed parameters to match 5.0 spec. Added test.

Jan 15 2019, 12:51 PM · Restricted Project

Jan 2 2019

tlwilmar added a comment to D55578: Add omp_get_device_num() and update several other device API functions (FIXED).

Is the intention to have this ready for the 8.0 release?

Jan 2 2019, 8:06 AM · Restricted Project

Dec 21 2018

tlwilmar accepted D55795: Fix for CMPLRS-48319 proc_bind affinity bug.

LGTM

Dec 21 2018, 11:54 AM · Restricted Project

Dec 18 2018

tlwilmar updated the diff for D55578: Add omp_get_device_num() and update several other device API functions (FIXED).

Tests added.

Dec 18 2018, 2:25 PM · Restricted Project

Dec 12 2018

tlwilmar added a comment to D55578: Add omp_get_device_num() and update several other device API functions (FIXED).

Since you're using it twice already, is it worth abstracting the "find the appropriate function using dlsym" logic into a function?
I imagine that there will be other user-interface functions which will also need this, so abstracting it sooner rather than later seems a good thing to do.

Dec 12 2018, 10:38 AM · Restricted Project

Dec 11 2018

tlwilmar created D55578: Add omp_get_device_num() and update several other device API functions (FIXED).
Dec 11 2018, 2:13 PM · Restricted Project

Nov 29 2018

tlwilmar added a comment to D54342: Add omp_get_device_num() and update several other device API functions.

After this patch, omp_get_num_devices() seems to wrongly return 0 even when devices are present.

OK, on my internal build these are showing up as weak symbols with nm. But on my LLVM build, not. Will look into the LLVM build to make sure it allows for weak attribute.

Nov 29 2018, 3:24 PM · Restricted Project
tlwilmar added a comment to D54342: Add omp_get_device_num() and update several other device API functions.

After this patch, omp_get_num_devices() seems to wrongly return 0 even when devices are present.

Nov 29 2018, 2:39 PM · Restricted Project
tlwilmar created D55078: Add omp_pause_resource* API.
Nov 29 2018, 2:16 PM · Restricted Project
tlwilmar added a comment to D54342: Add omp_get_device_num() and update several other device API functions.

This commit conflicts with libomptarget. libomptarget have the same functions and the code may be confused during execution which version of the functions should be executed. This conflict should be resolved somehow. Before that, I think, this patch must be reverted.

Nov 29 2018, 1:55 PM · Restricted Project

Nov 28 2018

tlwilmar created D55026: Task Affinity API.
Nov 28 2018, 1:55 PM · Restricted Project

Nov 27 2018

tlwilmar accepted D54948: Minor cleanup of debug code.

LGTM

Nov 27 2018, 11:59 AM
tlwilmar accepted D54950: Fixed possible array out of bound access.

LGTM

Nov 27 2018, 11:58 AM

Nov 9 2018

tlwilmar created D54342: Add omp_get_device_num() and update several other device API functions.
Nov 9 2018, 12:12 PM · Restricted Project

Nov 6 2018

tlwilmar accepted D53380: Implementation of OpenMP 5.0 mutexinoutset task dependency type..

LGTM

Nov 6 2018, 2:15 PM

Oct 23 2018

tlwilmar added a reviewer for D53422: Support clang compiling under windows-gnu and windows-msvc: jlpeyton.
Oct 23 2018, 5:45 AM

Aug 6 2018

tlwilmar accepted D49105: [OpenMP] Cleanup code.

LGTM

Aug 6 2018, 1:41 PM · Restricted Project

Jul 26 2018

tlwilmar accepted D49857: [OpenMP] Implement GOMP doacross compatibility.

LGTM

Jul 26 2018, 11:56 AM · Restricted Project
tlwilmar accepted D49859: [OpenMP] Add GOMP version symbols for OMP_4.5 API.

LGTM

Jul 26 2018, 11:48 AM · Restricted Project
tlwilmar accepted D49823: [OpenMP] Fix build errors when building with KMP_DEBUG_ADAPTIVE_LOCKS=1.

LGTM

Jul 26 2018, 11:45 AM · Restricted Project
tlwilmar accepted D49699: [OpenMP][Stats] Cleanup stats gathering code.

LGTM

Jul 26 2018, 11:41 AM · Restricted Project

Apr 11 2018

tlwilmar accepted D45327: Introduce GOMP_taskloop API.

LGTM

Apr 11 2018, 1:13 PM · Restricted Project
tlwilmar accepted D45527: [OpenMP] Fix affinity API for KMP_AFFINITY=none|compact|scatter.

LGTM

Apr 11 2018, 12:43 PM · Restricted Project

Mar 26 2018

tlwilmar accepted D41838: Add summarizeStats.py to tools directory.

LGTM

Mar 26 2018, 11:41 AM · Restricted Project

Mar 22 2018

tlwilmar accepted D44793: Fixed __kmpc_get_target_offload() to call initialization..

LGTM

Mar 22 2018, 11:45 AM

Mar 19 2018

tlwilmar accepted D44637: Fix for https://bugs.llvm.org/show_bug.cgi?id=36705.

LGTM

Mar 19 2018, 11:02 AM

Mar 16 2018

tlwilmar added inline comments to D44577: Read OMP_TARGET_OFFLOAD and provide API to access ICV.
Mar 16 2018, 12:52 PM
tlwilmar created D44577: Read OMP_TARGET_OFFLOAD and provide API to access ICV.
Mar 16 2018, 12:49 PM
tlwilmar updated subscribers of D44522: [Libomptarget] Full implementation of the target-offload-icv.

We have a patch coming with libomp-side addition of OMP_TARGET_OFFLOAD envirable plus an internal API so that libomptarget can get the value from libomp. Would it help if we get this patch out faster?

Mar 16 2018, 8:20 AM · Restricted Project

Feb 14 2018

tlwilmar updated the diff for D41914: Improve threadprivate implementation.

Updated comment to be more general.

Feb 14 2018, 2:12 PM

Jan 10 2018

tlwilmar created D41914: Improve threadprivate implementation.
Jan 10 2018, 12:53 PM

Jan 9 2018

tlwilmar added inline comments to D41831: Minor code cleanup.
Jan 9 2018, 12:51 PM
tlwilmar added inline comments to D41831: Minor code cleanup.
Jan 9 2018, 12:15 PM

Jan 8 2018

tlwilmar created D41836: Simplify __kmp_expand_threads.
Jan 8 2018, 1:46 PM
tlwilmar created D41831: Minor code cleanup.
Jan 8 2018, 12:20 PM

Dec 4 2017

tlwilmar created D40804: Trivial enum fix.
Dec 4 2017, 1:51 PM

Nov 29 2017

tlwilmar accepted D40624: Eliminate double printing of verbose affinity settings.

LGTM

Nov 29 2017, 3:01 PM · Restricted Project
tlwilmar added reviewers for D40374: Make kmp_r_sched_t into a union: hbae, omalyshe.
Nov 29 2017, 2:28 PM

Nov 27 2017

tlwilmar accepted D40340: Warning is emitted when tiles are requested but cannot be used.

LGTM

Nov 27 2017, 11:38 AM

Nov 22 2017

tlwilmar created D40374: Make kmp_r_sched_t into a union.
Nov 22 2017, 1:27 PM

Aug 29 2017

tlwilmar created D37275: Minor code cleanup of Klocwork issues.
Aug 29 2017, 2:08 PM

Aug 10 2017

tlwilmar added a comment to D36510: OMP_PROC_BIND: better spread.
Aug 10 2017, 8:05 AM · Restricted Project
tlwilmar added a comment to D36510: OMP_PROC_BIND: better spread.

So I think in summary, what you are doing is creating the place partitions with the ceil(P/T)-sized partitions evenly spread amongst the floor(P/T) partitions (where P is num places and T is num_threads). Is that right?

Aug 10 2017, 7:45 AM · Restricted Project

Aug 9 2017

tlwilmar added a comment to D36510: OMP_PROC_BIND: better spread.

Hi Paul,
Can you be more specific about how this "improves the way threads are spread across cores"? The OMP spec is very specific on exactly how the spread algorithm should work.
Thanks!
Terry

Aug 9 2017, 10:50 AM · Restricted Project

Jul 28 2017

tlwilmar accepted D36017: Move lock acquire/release functions to safer locations in task deque cleanup code .

LGTM

Jul 28 2017, 2:13 PM
tlwilmar created D36011: Remove BUILD_TV.
Jul 28 2017, 12:11 PM
tlwilmar created D36009: Add new envirable KMP_TEAMS_THREAD_LIMIT.
Jul 28 2017, 11:28 AM

Jul 27 2017

tlwilmar updated the diff for D35912: Fix implementation of OMP_THREAD_LIMIT.

Added a test for OMP_THREAD_LIMIT env, api, and general functionality. Note that this test, when running correctly, will emit a Warning and a Hint, because the limit squashes the requested number of threads.

Jul 27 2017, 1:40 PM