Page MenuHomePhabricator

saiislam (Saiyedul Islam)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 15 2019, 2:59 AM (44 w, 1 d)

Recent Activity

Thu, Sep 17

saiislam added inline comments to D87841: [openmp][libomptarget] Include header from LLVM source tree.
Thu, Sep 17, 11:28 AM · Restricted Project, Restricted Project
saiislam added a comment to D87841: [openmp][libomptarget] Include header from LLVM source tree.

AFAIK, all files which are required/should be available in libomptarget reside in llvm/Frontend/OpenMP/* , so may be an option can be provided for only-tarball users to get files only from this directory.
This kind of approach will also enforce people exporting stuff from clang to libomptarget, to put everything in one place.

Thu, Sep 17, 11:08 AM · Restricted Project, Restricted Project

Tue, Sep 15

saiislam added a comment to D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.

Ping.

Tue, Sep 15, 7:03 AM · Restricted Project

Thu, Sep 10

saiislam updated the diff for D87413: [OpenMP] Load plugins from same directory as the libomptarget.so.

Moved the code to load plugins to a helper function and before loop on plugins.

Thu, Sep 10, 11:07 AM · Restricted Project

Wed, Sep 9

saiislam added a comment to D87413: [OpenMP] Load plugins from same directory as the libomptarget.so.

First of two patches after splitting D73657

Wed, Sep 9, 1:16 PM · Restricted Project
saiislam requested review of D87413: [OpenMP] Load plugins from same directory as the libomptarget.so.
Wed, Sep 9, 1:15 PM · Restricted Project

Mon, Sep 7

saiislam added a comment to D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.

Ping.

Mon, Sep 7, 5:55 AM · Restricted Project

Thu, Aug 27

saiislam added a comment to D85214: [OpenMP] Ensure testing for versions 4.5 and default - Part 3.

@saiislam This change seems to have broken a buildbot, please take a look at your earliest convenience.

Before: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/35354

After: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/35355

Thu, Aug 27, 2:25 PM · Restricted Project
saiislam committed rGa1ac047b3453: [OpenMP] Fix a failing test after D85214 (authored by saiislam).
[OpenMP] Fix a failing test after D85214
Thu, Aug 27, 1:58 PM
saiislam committed rGff260ad0e014: [OpenMP] Ensure testing for versions 4.5 and default - Part 3 (authored by saiislam).
[OpenMP] Ensure testing for versions 4.5 and default - Part 3
Thu, Aug 27, 12:37 PM
saiislam closed D85214: [OpenMP] Ensure testing for versions 4.5 and default - Part 3.
Thu, Aug 27, 12:37 PM · Restricted Project
saiislam committed rGa1bdf8f545cf: [OpenMP] Ensure testing for versions 4.5 and default - Part 2 (authored by saiislam).
[OpenMP] Ensure testing for versions 4.5 and default - Part 2
Thu, Aug 27, 11:51 AM
saiislam closed D85150: [OpenMP] Ensure testing for versions 4.5 and default - Part 2.
Thu, Aug 27, 11:51 AM · Restricted Project

Wed, Aug 26

saiislam added inline comments to D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.
Wed, Aug 26, 12:27 PM · Restricted Project
saiislam updated the diff for D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.
  1. Reformarting
  2. Comments
  3. Reduced scope of specialized PrePostActionTy
Wed, Aug 26, 12:16 PM · Restricted Project

Mon, Aug 24

saiislam updated the diff for D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.
  1. Moved amdgcn specific functions to CGOpenMPAMDGCN.cpp
  2. Removed tautology condition
  3. Corrected case of local variables
  4. Restored original formatting
  5. Changed back declaration of emit kernel methods as private
  6. Added support of amdgcn specific PrePostActionTy implementation and its corresponding test cases
  7. Changed static line numbers in new test cases with regex
  8. Other small code corrections
Mon, Aug 24, 3:31 PM · Restricted Project

Aug 19 2020

saiislam added a comment to D86232: [NFC][OpenMP] Update description of OMPGridValues enums.

This can't be used in the libomptarget plugins or in the deviceRTL, as both are required to compile without the llvm sources available.

Current preferences appears to be to copy and paste the numbers between the two files. At the least, we should have a test outside of llvm & openmp trees that verifies the numbers are the same in each.

@jdoerfert can you remember who was strongly against #include between openmp and llvm? I'm failing to find the discussion, and the mail "Sharing enums between llvm and openmp" didn't reach a conclusion

Aug 19 2020, 11:31 AM · Restricted Project
saiislam requested review of D86232: [NFC][OpenMP] Update description of OMPGridValues enums.
Aug 19 2020, 10:39 AM · Restricted Project

Aug 18 2020

saiislam added a comment to D85150: [OpenMP] Ensure testing for versions 4.5 and default - Part 2.

Please let me know if there are any changes/suggestions.

Aug 18 2020, 11:07 AM · Restricted Project

Aug 17 2020

saiislam requested review of D86097: [OpenMP][AMDGCN] Generate global variables and attributes for AMDGCN.
Aug 17 2020, 11:55 AM · Restricted Project

Aug 13 2020

saiislam committed rGeaa341fbea96: [OpenMP] Ensure testing for versions 4.5 and default - Part 1 (authored by saiislam).
[OpenMP] Ensure testing for versions 4.5 and default - Part 1
Aug 13 2020, 12:37 AM
saiislam closed D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.
Aug 13 2020, 12:37 AM · Restricted Project
saiislam added a comment to D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.

Time for check-clang has increased from ~51 sec to ~107 sec after this patch.
Putting it here before committing/closing the review.

Aug 13 2020, 12:36 AM · Restricted Project

Aug 11 2020

saiislam added a comment to D85214: [OpenMP] Ensure testing for versions 4.5 and default - Part 3.

Ping.

Aug 11 2020, 11:36 PM · Restricted Project
saiislam added a comment to D85150: [OpenMP] Ensure testing for versions 4.5 and default - Part 2.

Ping.

Aug 11 2020, 11:36 PM · Restricted Project
saiislam added a comment to D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.

Ping for this (part-1), D85150 (part-2), and D85214 (part-3).

Aug 11 2020, 6:33 AM · Restricted Project

Aug 4 2020

saiislam added a comment to D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.

Next two (independent) parts are: D85150 and D85214. More to come.

Aug 4 2020, 8:27 AM · Restricted Project
saiislam abandoned D82575: [OpenMP] OpenMP Clang tests without 50 version string after upgrading to 5.0.

Abandoning in favor of D85214

Aug 4 2020, 8:25 AM · Restricted Project
saiislam added a comment to D85214: [OpenMP] Ensure testing for versions 4.5 and default - Part 3.

This patch updates 101 out 320 test files. Remaining will be posted as a separate patch to ease review.

Aug 4 2020, 8:25 AM · Restricted Project
saiislam requested review of D85214: [OpenMP] Ensure testing for versions 4.5 and default - Part 3.
Aug 4 2020, 8:23 AM · Restricted Project

Aug 3 2020

saiislam abandoned D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version.

Abandoning in favor of D84844 and D85150 i.e. adding default version testing along with version 4.5 testing.

Aug 3 2020, 12:48 PM · Restricted Project
saiislam requested review of D85150: [OpenMP] Ensure testing for versions 4.5 and default - Part 2.
Aug 3 2020, 12:44 PM · Restricted Project
saiislam retitled D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1 from [OpenMP] Ensure testing for versions 4.5, 5.0, and default - Part 1 to [OpenMP] Ensure testing for versions 4.5 and default - Part 1.
Aug 3 2020, 10:46 AM · Restricted Project
saiislam updated the diff for D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.

Removed redundant testing for version 5.0 with its version string.

Aug 3 2020, 10:45 AM · Restricted Project

Aug 2 2020

saiislam committed rG160ff83765ac: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3 (authored by saiislam).
[OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3
Aug 2 2020, 10:55 PM
saiislam closed D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.
Aug 2 2020, 10:55 PM · Restricted Project

Jul 30 2020

saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Changes as requested.

Jul 30 2020, 11:51 AM · Restricted Project
saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Improved comment placement. Corrected check lines. Rebased.

Jul 30 2020, 11:30 AM · Restricted Project
saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Moved comments to header files. Used regexps for test cases.

Jul 30 2020, 8:13 AM · Restricted Project

Jul 29 2020

saiislam requested review of D84844: [OpenMP] Ensure testing for versions 4.5 and default - Part 1.
Jul 29 2020, 7:02 AM · Restricted Project

Jul 28 2020

saiislam requested review of D84743: [Clang][AMDGCN] Universal device offloading macros header.
Jul 28 2020, 5:23 AM · Restricted Project

Jul 27 2020

saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Generalized regex in one of the test to pass harbormaster build.

Jul 27 2020, 1:12 AM · Restricted Project

Jul 24 2020

saiislam added inline comments to D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.
Jul 24 2020, 10:50 PM · Restricted Project
saiislam added a reviewer for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3: arsenm.
Jul 24 2020, 10:46 PM · Restricted Project
saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Added InitTempAlloca address space cast.

Jul 24 2020, 10:43 PM · Restricted Project
saiislam added a comment to D83730: [AMDGPU] Spill CSR VGPR which is reserved for SGPR spills.

LGTM. Please wait for @arsenm 's review.

Jul 24 2020, 12:24 AM · Restricted Project

Jul 21 2020

saiislam updated the diff for D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3.

Added final to specialized classes.

Jul 21 2020, 12:40 PM · Restricted Project
Herald added a project to D84260: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 3: Restricted Project.
Jul 21 2020, 12:04 PM · Restricted Project
saiislam accepted D84221: [OpenMP] Add missing RUN lines for OpenMP 4.5.

Thank you. LGTM.

Jul 21 2020, 8:39 AM · Restricted Project
saiislam committed rG741e55aeed75: [OpenMP] Temporarily disable failing runtime tests for clang-12 (authored by saiislam).
[OpenMP] Temporarily disable failing runtime tests for clang-12
Jul 21 2020, 8:33 AM
saiislam closed D84241: [OpenMP] Temporarily disable failing runtime tests for clang-12.
Jul 21 2020, 8:33 AM · Restricted Project
saiislam added a comment to D84241: [OpenMP] Temporarily disable failing runtime tests for clang-12.

It is causing various recent OpenMP harbormaster builds to fail like the one for D84148 and D84221.

Jul 21 2020, 8:25 AM · Restricted Project
Herald added a project to D84241: [OpenMP] Temporarily disable failing runtime tests for clang-12: Restricted Project.
Jul 21 2020, 8:22 AM · Restricted Project

Jul 20 2020

saiislam added a comment to D84221: [OpenMP] Add missing RUN lines for OpenMP 4.5.

Can you add test with version string 50 also?
So, tests should be there for version 45, version 50, and default version. It will ensure that this test doesn't silently stops testing for 50 when we upgrade to the next version.

Jul 20 2020, 11:21 PM · Restricted Project
saiislam committed rGfc7d2908ab38: [OpenMP] Use common interface to access GPU Grid Values (authored by saiislam).
[OpenMP] Use common interface to access GPU Grid Values
Jul 20 2020, 10:26 PM
saiislam closed D83492: [OpenMP] Use common interface to access GPU Grid Values.
Jul 20 2020, 10:26 PM · Restricted Project
saiislam added a comment to D64571: [OPENMP]Fix threadid in __kmpc_omp_taskwait call for dependent target calls..

The test fails for me with segfault in line 22.

$ gdb target_depend_nowait.cpp.tmp-x86_64-pc-linux-gnu
(gdb) run
...
Program received signal SIGSEGV, Segmentation fault.
(gdb) bt
#0  0x0000000000400fcf in .omp_outlined._debug__ (.global_tid.=0x2aaab96b9be0, .bound_tid.=0x2aaab96b9bd8) at target_depend_nowait.cpp:22
#1  0x0000000000401e8d in .omp_outlined..23 (.global_tid.=0x2aaab96b9be0, .bound_tid.=0x2aaab96b9bd8) at target_depend_nowait.cpp:18
#2  0x00002aaaab574213 in __kmp_invoke_microtask () from libomp.so
#3  0x00002aaaab51338e in __kmp_invoke_task_func () from libomp.so
#4  0x00002aaaab5126bf in __kmp_launch_thread () from libomp.so
#5  0x00002aaaab55d3d0 in __kmp_launch_worker(void*) () from libomp.so
#6  0x00002aaaabbd6e65 in start_thread () from libpthread.so.0
#7  0x00002aaaabee988d in clone () from libc.so.6

It is not the problem of this patch, looks like support for mappers causes failures

Jul 20 2020, 8:39 AM · Restricted Project, Restricted Project

Jul 17 2020

saiislam updated the diff for D83492: [OpenMP] Use common interface to access GPU Grid Values.

Corrected.

Jul 17 2020, 9:07 AM · Restricted Project
saiislam updated the diff for D83492: [OpenMP] Use common interface to access GPU Grid Values.

Removed getNVPTXWarpSize() changes for a separate patch and rebased.

Jul 17 2020, 8:43 AM · Restricted Project
saiislam added inline comments to D83492: [OpenMP] Use common interface to access GPU Grid Values.
Jul 17 2020, 8:39 AM · Restricted Project
saiislam committed rGc7562e77b3ac: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU (authored by saiislam).
[OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU
Jul 17 2020, 7:39 AM
saiislam closed D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.
Jul 17 2020, 7:39 AM · Restricted Project

Jul 15 2020

saiislam added inline comments to D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version.
Jul 15 2020, 6:28 AM · Restricted Project
saiislam added a comment to D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version.

Not sure that this is really required. If we want to check for the default version, it would be good to check that it still works for 4.5. It means, need to add additional RUN lines. But I don't think it is profitable to test the same functionality several times without actual effect

These 67 files were testing only for version 4.5. As the default version is now 5.0, isn't it expected that the tests run against it?

It just means that there are no changes in functionality between 4.5 and 5.0. Sure, you can switch to the default version instead but I don't see real profit here. Anyway, it is up to you.

Jul 15 2020, 5:54 AM · Restricted Project
saiislam added a comment to D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version.

Not sure that this is really required. If we want to check for the default version, it would be good to check that it still works for 4.5. It means, need to add additional RUN lines. But I don't think it is profitable to test the same functionality several times without actual effect

Jul 15 2020, 5:42 AM · Restricted Project
saiislam added a comment to D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version.

[Work in progress]
Among all the test files in clang/test/OpenMP which were running only for version 4.5, I am yet to fix following files (rest all are given in this patch):

Jul 15 2020, 5:00 AM · Restricted Project
Herald added a project to D83863: [OpenMP] Change version 4.5 hardcoded clang tests to default OpenMP version: Restricted Project.
Jul 15 2020, 4:53 AM · Restricted Project

Jul 14 2020

saiislam updated the summary of D82575: [OpenMP] OpenMP Clang tests without 50 version string after upgrading to 5.0.
Jul 14 2020, 11:56 PM · Restricted Project
saiislam updated the diff for D82575: [OpenMP] OpenMP Clang tests without 50 version string after upgrading to 5.0.

Instead of adding additional test cases without version string, removed all occurrences of -fopenmp-version=50.

Jul 14 2020, 11:44 PM · Restricted Project
saiislam added a comment to D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.

Thank you.
I will hold committing till llvm-11 branching so it doesn't break something in any downstream dependent project at the last moment.

Jul 14 2020, 12:59 PM · Restricted Project
saiislam updated the summary of D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.
Jul 14 2020, 7:19 AM · Restricted Project
saiislam updated the diff for D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.

Removed all references to AMDGCN to make it NFC. Will introduce AMDGCN specific things separately in a followup patch.

Jul 14 2020, 7:18 AM · Restricted Project
saiislam updated the diff for D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.

Removed all formatting and spacing changes introduced due to clang-format.

Jul 14 2020, 1:42 AM · Restricted Project

Jul 13 2020

saiislam updated subscribers of D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU.
Jul 13 2020, 3:04 PM · Restricted Project
saiislam added a comment to D83492: [OpenMP] Use common interface to access GPU Grid Values.

Changing to getGridValue would be useful for sharing parts of this with amdgcn.

The aomp toolchain handles codegen for amdgcn by adding if (isAMDGCN) to this file. Until such time as tregions obsoletes this code, I think we should go with layers instead of scattered conditionals.

I.e. rename CGOpenMPRuntimeNVPTX to CGOpenMPRuntimeGPU which contains code that is common to nvptx and amdgcn. That probably uses getGridValue() as a way to abstract over minor differences. Derive CGOpenMPRuntimeAMDGCN and CGOpenMPRuntimeNVPTX from CGOpenMPRuntimeGPU to implement (virtual) functions which are different between the two.

Jul 13 2020, 2:55 PM · Restricted Project
Herald added a project to D83723: [OpenMP][NFC] Generalize CGOpenMPRuntimeNVPTX as CGOpenMPRuntimeGPU: Restricted Project.
Jul 13 2020, 2:49 PM · Restricted Project

Jul 9 2020

Herald added a project to D83492: [OpenMP] Use common interface to access GPU Grid Values: Restricted Project.
Jul 9 2020, 9:56 AM · Restricted Project

Jul 6 2020

saiislam committed rG38d6640ba55e: [libomptarget] Implement atomic inc and fence functions for AMDGCN using clang… (authored by saiislam).
[libomptarget] Implement atomic inc and fence functions for AMDGCN using clang…
Jul 6 2020, 11:37 PM
saiislam committed rG0882c9d4fc49: [AMDGPU] Change Clang AMDGCN atomic inc/dec builtins to take unsigned values (authored by saiislam).
[AMDGPU] Change Clang AMDGCN atomic inc/dec builtins to take unsigned values
Jul 6 2020, 11:37 PM
saiislam closed D83132: [libomptarget] Implement atomic inc and fence functions for AMDGCN using clang builtins.
Jul 6 2020, 11:37 PM · Restricted Project
saiislam closed D83121: [AMDGPU] Change Clang AMDGCN atomic inc/dec builtins to take unsigned values.
Jul 6 2020, 11:37 PM · Restricted Project
saiislam committed rG4c4bda1630a6: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0 (authored by saiislam).
[OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0
Jul 6 2020, 7:05 AM
saiislam closed D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.
Jul 6 2020, 7:05 AM · Restricted Project

Jul 3 2020

saiislam updated the diff for D83132: [libomptarget] Implement atomic inc and fence functions for AMDGCN using clang builtins.
  1. Removed unused functions
  2. Changed to INLINE
  3. Moved fence functions to target_impl.h
Jul 3 2020, 10:12 AM · Restricted Project
saiislam created D83132: [libomptarget] Implement atomic inc and fence functions for AMDGCN using clang builtins.
Jul 3 2020, 8:02 AM · Restricted Project
saiislam added a comment to D80804: [AMDGPU] Introduce Clang builtins to be mapped to AMDGCN atomic inc/dec intrinsics.

This patch declares the clang builtin as acting on signed values, but the IR intrinsic maps to an instruction which does an unsigned comparison. We don't have ISA support for a signed comparison equivalent. Addition is the same operation on signed or unsigned integers, but signed integer comparison is not equivalent to unsigned integer comparison.

// 32bit
 tmp = MEM[ADDR];
 MEM[ADDR] = (tmp >= DATA) ? 0 : tmp + 1; // unsigned
compare
 RETURN_DATA = tmp.

The builtins should be changed to take unsigned values, optionally making that clear from the naming scheme, perhaps __amdgcn_builtin_atomic_dec_u32.

Apologies for not reviewing this the first time around.

Jul 3 2020, 3:12 AM · Restricted Project
saiislam created D83121: [AMDGPU] Change Clang AMDGCN atomic inc/dec builtins to take unsigned values.
Jul 3 2020, 3:12 AM · Restricted Project
saiislam added a comment to D83077: [OpenMP][Tests] Flag compatibility of OpenMP runtime tests with GCC versions.

I have no idea, what Harbormaster is trying to tell me.

https://buildkite.com/llvm-project/premerge-checks/builds/2535#_

shows a failure in ninja all, but the only information about the command is in the artifact, which shows [4042/4042] successfully built targets.

Jul 3 2020, 3:12 AM · Restricted Project

Jul 2 2020

saiislam updated the diff for D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.

Removed fix for OpenMPTesting.cmake bug.

Jul 2 2020, 11:19 AM · Restricted Project
saiislam retitled D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0 from [OpenMP] Temporarily disable failing runtime and ompt tests for OpenMP 5.0 to [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.
Jul 2 2020, 11:19 AM · Restricted Project
saiislam added a comment to D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.

Thanks @Hahnfeld. I realized that LLVM_MAJOR_VERSION was neither getting set in OpenMPTesting.cmake nor was it
inheriting it from anywhere else. So, OPENMP_TEST_COMPILER_VERSION_MAJOR was also getting set as empty, which
was getting propagated to lit by config.test_compiler_features. That is why "clang-11" was not getting recognized
as a valid target by lit-unsupported (though clang-11.0.0 would have worked). This change should fix this issue.

That's because the variable is called LLVM_VERSION_MAJOR (and it's been wrong since I added that code in 2017). I went ahead and fixed that mistake in https://github.com/llvm/llvm-project/commit/0e0483bf5c383d5815b9f945fea7e347d4badc0e. You shouldn't have to deal with broken infrastructure in this patch :)

Jul 2 2020, 10:47 AM · Restricted Project
saiislam added a comment to D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.

Does clang-11 now default to 50 behavior?
In this case, we can remove -fopenmp-version=50 from the tests, right?

Jul 2 2020, 10:47 AM · Restricted Project
saiislam updated the diff for D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.

Thanks @Hahnfeld. I realized that LLVM_MAJOR_VERSION was neither getting set in OpenMPTesting.cmake nor was it
inheriting it from anywhere else. So, OPENMP_TEST_COMPILER_VERSION_MAJOR was also getting set as empty, which
was getting propagated to lit by config.test_compiler_features. That is why "clang-11" was not getting recognized
as a valid target by lit-unsupported (though clang-11.0.0 would have worked). This change should fix this issue.

Jul 2 2020, 9:42 AM · Restricted Project
saiislam accepted D82267: [OpenMP][Tests] Fix/Mark compatibilty for GCC.

LGTM. Thank you!
Please link any ongoing/future effort to enable support for gcc-9/10 for the failing tests, as and when it materializes.

Jul 2 2020, 8:05 AM · Restricted Project
saiislam updated the diff for D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.
  1. Added clang version identification in openmp/runtime/test
  2. Marked these three tests as unsupported for clang-11
  3. Refactored test lines to pass clang-format
Jul 2 2020, 4:48 AM · Restricted Project

Jul 1 2020

saiislam added inline comments to D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.
Jul 1 2020, 11:54 AM · Restricted Project
saiislam added a comment to D81098: [OpenMP] Upgrade default version of OpenMP to 5.0.

LG

Since this revision landed two tests are failing.

libomp.env::kmp_set_dispatch_buf.c and libomp.worksharing/for::kmp_set_dispatch_buf.c. It was also reported on the mailing list (http://lists.llvm.org/pipermail/openmp-dev/2020-June/003507.html). Any idea how we can fix this quickly? @jdoerfert
OR maybe this is known and will be taken care later?

Jul 1 2020, 8:05 AM · Restricted Project
saiislam added a reviewer for D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0: clementval.
Jul 1 2020, 8:05 AM · Restricted Project
saiislam created D82963: [OpenMP] Temporarily disable failing runtime tests for OpenMP 5.0.
Jul 1 2020, 7:32 AM · Restricted Project
saiislam updated the diff for D82575: [OpenMP] OpenMP Clang tests without 50 version string after upgrading to 5.0.

[Work in progress] 48 files inspected/corrected. Following 35 are yet to be checked:

Jul 1 2020, 7:00 AM · Restricted Project