Compiler Engineer at AMD
User Details
- User Since
- Dec 24 2019, 1:39 AM (161 w, 5 d)
Tue, Jan 17
Please add a test case which would fail with the old implementation but passes with the new one.
Oct 13 2022
Sep 22 2022
Rebase with main. Thanks for the review @raghavendhra.
Ping for review!
Sep 15 2022
Thanks for the patch @SouraVX. LGTM.
Sep 13 2022
Ping for review.
Sep 4 2022
Rebase with main. Adressed comment.
Aug 31 2022
Aug 30 2022
Addressed review comment.
Addressed review comment.
Rebase with main
Aug 25 2022
Apologies for the delay in review. Please let me know what you think. I will join the next OpenMP call and we can maybe discuss this.
Aug 15 2022
Aug 14 2022
Minor comments. Thank you for working on this!
Aug 13 2022
Ping for review.
Aug 9 2022
Jul 29 2022
LGTM, minor comments. Please let me know what you think!
Jul 28 2022
Removed NoSideEffect.
Rebase with main
Jul 27 2022
LGTM.
Addressed comments.
Jul 26 2022
I have reverted this change for now, we can discuss and add TODOs for the unhandled cases and then merge it.
Jul 25 2022
Jul 22 2022
Just a thought - for the long run, would it make sense to have OpenMPConverter class inheriting from FirConverter and overriding some functions- like genFIR for OpenMP constructs and genFIR for assignment statement? IMO this would make the file more structured, while separating OpenMP and Fortran code. Right now, OpenMP.cpp is just a bunch of functions.
Jul 21 2022
Ping for review. (This is not urgent for the drop on July 26th, and patches related to those are high priority. This is just a gentle reminder, if anyone is willing to look into this patch. :) )
Jul 20 2022
Rebase with main
Jul 19 2022
Nevermind, this has been fixed in https://github.com/llvm/llvm-project/commit/2c488a6b35c6d03e34fdd4e7ec87ed6bdc3d3010
Jul 18 2022
Moved checking to isSupportedByOpenMPIRBuilder.
Rebase with main
Jul 14 2022
Any pointer to this presentation?
This was discussed today in the OpenMP work for llvm-flang call.
Jul 12 2022
Jul 11 2022
Jul 10 2022
Ping for review.
Jul 6 2022
Jul 5 2022
LGTM. Thank you!
Jul 4 2022
The generated code in testcases looks good. Please wait for @peixin's approval for the code part, but functionality-wise this LGTM.
Jun 29 2022
Jun 28 2022
Jun 27 2022
Hmm, thats interesting. This makes me doubt if we need the barrier at all for firstprivate? When do we need such a barrier? Should we add it only when #pragma omp barrier is allowed inside a region? Maybe all such testcases are "bad" and we would be fine without the barrier too? We should have a discussion and maybe ask the OpenMP experts about this.
If there are no further issues, I will merge this tomorrow. It would be nice to have E2E task support.
Ping for review!
Jun 26 2022
Hi @peixin thanks for this. In the following, we need some sort of synchronization in this construct to make sure both the regions get their copies before they start executing. How should we handle this?
Jun 21 2022
Addressed comments.