- User Since
- Apr 16 2015, 7:54 AM (301 w, 16 h)
Tue, Jan 19
Mon, Jan 18
Tue, Jan 12
Wed, Jan 6
Wed, Dec 30
Dec 15 2020
Dec 9 2020
Dec 7 2020
Dec 1 2020
Nov 30 2020
Updated the style of newly added function.
Add function for reading numeric only values from location string. Use the function in metadata reporting for loop and single constructs.
Nov 25 2020
Nov 24 2020
Patch would still better be updated with correct path and full context (maybe post commit).
Nov 23 2020
I am sorry, my accept action was not formal enough.
Nov 20 2020
Nov 19 2020
Nov 18 2020
Note that I assume that hitting a file we left behind by ourselves is not a problem for the runtime.
You will still need to explicitly ask library to work with files left behind via setting KMP_DUPLICATE_LIB_OK=1.
Otherwise library should abort, as it will think that another copy of the library already works in the same process, that is not supported.
Nov 17 2020
Nov 16 2020
Please also change the openmp/runtime/tools/generate-def.pl file in order to not break Windows build.
See https://reviews.llvm.org/D91478 as an example of the change made for omp_calloc function.
Nov 15 2020
Nov 13 2020
Nov 12 2020
Nov 10 2020
I mean for example __kmp_str_loc_init, which does gracefully handle ill-formed input (as far as I can tell).
If it doesn't, let's fix it as well. Either way, we should probably reuse it.
Nov 9 2020
Nov 6 2020
Nov 4 2020
Oct 26 2020
Oct 23 2020
Accepting as there were no objections.
I cannot check the correctness myself as don't have access to these architectures.
Sep 15 2020
Sep 14 2020
Sep 11 2020
Looks good, besides a few formatting nits.
Sep 9 2020
Couple of tests needed to check if the implementation works - one with unshackled task encountered before parallel, and another with unshackled task encountered after / between parallels.
Aug 26 2020
This change causes standalone Windows build to fail:
Aug 24 2020
Aug 19 2020
Aug 17 2020
The key difference is:
git diff # gives you short patch
git diff -U999999 # gives you full patch
The full patch should have complete source, as opposed to 3 lines above and below the changes for the short patch.
Ugh, sorry I missed that! Fixed.
Aug 12 2020
Aug 11 2020
Could you please provide full diff so that more context is available around the changes?
Aug 5 2020