guansong (Guansong Zhang)
User

Projects

User does not belong to any projects.

User Details

User Since
May 14 2014, 8:22 AM (218 w, 1 d)

Recent Activity

Jun 19 2018

guansong committed rOMP335046: [OpenMP] [CUDA] Expose teamid to the debug path.
[OpenMP] [CUDA] Expose teamid to the debug path
Jun 19 2018, 8:10 AM
guansong committed rL335046: [OpenMP] [CUDA] Expose teamid to the debug path.
[OpenMP] [CUDA] Expose teamid to the debug path
Jun 19 2018, 7:10 AM
guansong closed D48286: [OpenMP] [CUDA] Expose teamid to the debug path.
Jun 19 2018, 7:10 AM · Unknown Object (Project)

Jun 18 2018

guansong created D48286: [OpenMP] [CUDA] Expose teamid to the debug path.
Jun 18 2018, 9:30 AM · Unknown Object (Project)

May 29 2018

guansong added a comment to D47394: [OpenMP][Clang][NVPTX] Replace bundling with partial linking for the OpenMP NVPTX device offloading toolchain.

I am not quite familiar with Clang driver set up, I will add Greg for more comments. But I have hacked one the latest YKT tree to support simple AMDGCN path the same way as NVPTX. The last patch is here

May 29 2018, 9:06 AM

May 18 2018

guansong added a comment to D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available .

Can we double check this LLVM IR backward compatible thing? My understanding is that it is not that good, at least for the major versions.

I just want to comment this last point, my understanding is that it will not work even you try to inline older LLVM IR into a newer one.

I did just that: I built libomptarget-nvptx-bc with Clang 3.9.1 (which is actually the only released version that you can currently build with) and inlined the result with LLVM/Clang trunk which will become 7.0. It worked without problems for all simple test cases I tried. Do you have a scenario that fails for you?

May 18 2018, 1:37 AM
guansong added a comment to D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available .

I am not at all convinced. I think there are absolutely no good arguments against allowing the just built compiler to build the BCLIB (as a default, this is simply a default which can be overwritten at any time by the user). The drawbacks you mention have no detrimental impact on the quality of life of the compiler developer. Developers improving other parts of the compiler who do not care about OpenMP and device offloading can just disable the building of the BCLIB because it is optional after all. The BCLIB is something that is required for performance not correctness so it can be disabled at any time. Even when BCLIB building is turned on there is no perceptible overhead to it, compared to the time it takes to compile a cpp file somewhere in the source code, the BCLIB builds instantly.

Your argument that an older version is better is actually wrong. Using an older version to build the BCLIB is always a BAD idea. The reason why you want to build the BCLIB is so that the runtime functions actually get inlined. If you use an older compiler the functions will not be inlined which defeats the purpose of having the BCLIB in the first place. I stumbled again upon this problem today after updating to the latest changes.

I do not understand why you are opposing this, so far I haven't heard a single strong argument against this.

May 18 2018, 1:25 AM

May 15 2018

guansong added a comment to D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available .
May 15 2018, 7:16 AM
guansong added a comment to D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available .

Can we double check this LLVM IR backward compatible thing? My understanding is that it is not that good, at least for the major versions.

May 15 2018, 7:01 AM
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

Thanks for the response. Can you point me to the final solution for this or re-explain it? From the comments I'm not sure I can distill the solution you want.

May 15 2018, 4:51 AM · Unknown Object (Project)

May 4 2018

guansong committed rOMP331550: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on….
[OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on…
May 4 2018, 12:41 PM
guansong committed rL331550: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on….
[OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on…
May 4 2018, 12:33 PM
guansong closed D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.
May 4 2018, 12:33 PM · Unknown Object (Project)
guansong added a comment to D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.
May 4 2018, 10:07 AM · Unknown Object (Project)
guansong updated the diff for D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.

Update as suggested.

May 4 2018, 10:06 AM · Unknown Object (Project)

May 3 2018

guansong updated the diff for D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.

Update name as suggested.

May 3 2018, 8:03 PM · Unknown Object (Project)

May 2 2018

guansong added a comment to D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.

May I suggest that we use something like LIBOMPTARGET_DEVICE_RTL_DEBUG instead of LIBOMPDEVICE_DEBUG? I think this name is more descriptive of what the env var does.

May 2 2018, 8:53 AM · Unknown Object (Project)

Apr 27 2018

guansong created D46210: [OpenMP] Use LIBOMPTARGET_DEVICE_RTL_DEBUG env var to control debug messages on the device side.
Apr 27 2018, 2:22 PM · Unknown Object (Project)
guansong added a comment to D46185: [OpenMP] Allow nvptx sm_30 to be used as an offloading device.

Anyway, why do you want to support sm_30? Is there a particular hardware that you want to support? Why does AMD care?!?

I will double check with Greg on the way atomicCAS used in the code. For the last question here. We have sm_30 machine for testing. Besides, from my perspective, I care the overall implementation of openmp, particularly the offloading part, not necessarily amdgcn code only :)

Okay, then the question is: Will the implementation get better if it supports 6 years old hardware with yet another code path that only few people care about and that will get less testing? From intuition I doubt that because extra code for old hardware won't get much support and attention by definition.

Apr 27 2018, 11:11 AM · Unknown Object (Project)
guansong added a comment to D46185: [OpenMP] Allow nvptx sm_30 to be used as an offloading device.

I don't think this code does atomic max. Actually, I don't think it does anything at all. First off, old_value and elem are variables which do not change inside the while-loop. So the first loop condition is either always true or always false. Secondly, atomicCAS returns the value that existed at address Buffer (regardless of whether or not the CAS operation succeeded) and not a boolean result. Also, old_value must be updated before each CAS attempt because some other thread may have changed it.

A correct implementation would look somewhat like this:

uint64_t old_value = *Buffer;
uint64_t expected_old_value;
do {
  expected_old_value = old_value;
  old_value = atomicCAS((unsigned long long *) Buffer,
                        (unsigned long long) expected_old_value,
                        (unsigned long long) max(expected_old_value, elem));
} while (expected_old_value != old_value);

What this code does is: check that the value at Buffer is still what I expect it to be, i.e. no other thread has changed it, and replace it with max(old_value, elem).

Fully agree with George's analysis here. The proposed implementation may be tweaked to early exit if elem < *Buffer.

Anyway, why do you want to support sm_30? Is there a particular hardware that you want to support? Why does AMD care?!?

Apr 27 2018, 10:46 AM · Unknown Object (Project)
guansong added a reviewer for D46185: [OpenMP] Allow nvptx sm_30 to be used as an offloading device: gregrodgers.
Apr 27 2018, 6:08 AM · Unknown Object (Project)
guansong created D46185: [OpenMP] Allow nvptx sm_30 to be used as an offloading device.
Apr 27 2018, 6:02 AM · Unknown Object (Project)

Apr 26 2018

guansong committed rOMP330944: [OpenMP] Remove compilation warning when using clang to compile bc files..
[OpenMP] Remove compilation warning when using clang to compile bc files.
Apr 26 2018, 7:34 AM
guansong committed rL330944: [OpenMP] Remove compilation warning when using clang to compile bc files..
[OpenMP] Remove compilation warning when using clang to compile bc files.
Apr 26 2018, 7:10 AM
guansong closed D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..
Apr 26 2018, 7:10 AM · Unknown Object (Project)

Apr 25 2018

guansong updated the diff for D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

Use PRIu64 for chuck

Apr 25 2018, 9:21 PM · Unknown Object (Project)
guansong updated the diff for D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

With the proper position of include file, I can use PRId64 now.

Apr 25 2018, 9:11 PM · Unknown Object (Project)

Apr 20 2018

guansong committed rOMP330477: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
[OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag
Apr 20 2018, 1:45 PM
guansong committed rL330477: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
[OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag
Apr 20 2018, 1:44 PM
guansong closed D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
Apr 20 2018, 1:44 PM · Unknown Object (Project)

Apr 19 2018

guansong added a comment to D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

I checked again on the format you suggested. To use it, you need #include <inttypes.h> For example the macro was defined like this in my version of the inttypes.h

# if __WORDSIZE == 64
#  define __PRI64_PREFIX        "l"
#  define __PRIPTR_PREFIX       "l"
# else
#  define __PRI64_PREFIX       "ll"
#  define __PRIPTR_PREFIX
# endif

# define PRId8 "d"
# define PRId16 "d"
# define PRId32 "d"
# define PRId64 __PRI64_PREFIX "d"

But the header causes a compilation issue for cuda clang. I am not sure how to solve that problem.

Apr 19 2018, 12:49 PM · Unknown Object (Project)
guansong added a comment to D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

I checked again on the format you suggested. To use it, you need #include <inttypes.h> For example the macro was defined like this in my version of the inttypes.h

Apr 19 2018, 12:20 PM · Unknown Object (Project)

Apr 15 2018

guansong added a comment to D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

update as suggested (except PRId64)

Isn't PRId64 working?

I did not find any usage of it on the device side. Not confident on using it. We can remove the warning first, and improve the format later?

It's not giving a warning because in CUDA long long int happens to be 64 bits long, so you can use %lld to print an int64_t. This is not guaranteed to be true on every platform, therefore the portable way to print an int64_t is using the PRId64 macro. That's what we do in the base library.

Apr 15 2018, 10:26 AM · Unknown Object (Project)

Apr 13 2018

guansong added a comment to D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

update as suggested (except PRId64)

Isn't PRId64 working?

Apr 13 2018, 2:49 PM · Unknown Object (Project)
guansong updated the diff for D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..

update as suggested (except PRId64)

Apr 13 2018, 11:54 AM · Unknown Object (Project)

Apr 12 2018

guansong abandoned D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.
Apr 12 2018, 2:17 PM · Unknown Object (Project)
guansong added a comment to D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.

Ok, let me abandon this one and think on this again.

Apr 12 2018, 2:13 PM · Unknown Object (Project)

Apr 11 2018

guansong updated the diff for D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.

update according to suggestion, thanks.

Apr 11 2018, 1:55 PM · Unknown Object (Project)
guansong added inline comments to D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
Apr 11 2018, 11:05 AM · Unknown Object (Project)
guansong added inline comments to D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
Apr 11 2018, 11:00 AM · Unknown Object (Project)
guansong retitled D45528: [OpenMP] Remove compilation warning when using clang to compile bc files. from Remove compilation warning when using clang to compile bc files. to [OpenMP] Remove compilation warning when using clang to compile bc files..
Apr 11 2018, 10:28 AM · Unknown Object (Project)
guansong retitled D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag from Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag to [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
Apr 11 2018, 10:28 AM · Unknown Object (Project)
guansong created D45530: [OpenMP] Make bc file compilation sensitive to LIBOMPTARGET_NVPTX_DEBUG flag.
Apr 11 2018, 10:28 AM · Unknown Object (Project)
guansong created D45528: [OpenMP] Remove compilation warning when using clang to compile bc files..
Apr 11 2018, 10:21 AM · Unknown Object (Project)

Apr 10 2018

guansong committed rOMP329713: [OpenMP] Remove extra warning when we build.
[OpenMP] Remove extra warning when we build
Apr 10 2018, 8:32 AM
guansong committed rL329713: [OpenMP] Remove extra warning when we build.
[OpenMP] Remove extra warning when we build
Apr 10 2018, 8:31 AM
guansong closed D45415: [OpenMP] Remove extra warning when we build.
Apr 10 2018, 8:31 AM · Unknown Object (Project)

Apr 9 2018

guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

! In D44992#1061781, @guansong wrote:
I have revert the change, let's continue to discuss how to enable bc file path better.

You are still not subscribed to openmp-commits, so again no commit email! For reference, the commit is rL329576 (and you shouldn't quote git commit hashes, there is a script in LLVM to revert SVN changes from git-svn.)

I was trying to use my office email to subscribe the mailing list, it may not be approved yet. How can I check on that? Currently I only get openmp-commits message from my gmail.

For reverting, I revert the patch from git, and commit it use git svn dcommit. I guess you are referring the following comment in the log

This reverts commit 6849e31c36d712d97433bca9af39b7a09c8c1207.

generated during the process. What is the proper script to use?

There is utils/git-svn/git-svnrevert in the main LLVM repository.

I can see the advantage of dynamic testing. But to test everything dynamically maybe tedious. For bc file handling, I consider this is more of a project need to embed LLVM itself, using find_package is more suitable. http://llvm.org/docs/CMake.html#embedding

proper link: http://llvm.org/docs/CMake.html#embedding-llvm-in-your-project

We probably don't need to test all flags on their own, we could group the basic flags together. And I still don't get which variable you want to use from that CMake integration?

Unless the grouped flags get accepted or rejected together by the compiler, we won't know which flag caused the failure. For the moment, I intend to use LLVM_VERSION_MAJOR and LLVM_INSTALL_PREFIX/LLVM_BUILD_BINARY_DIR only. But I can see more if we further explore the bc files.

I'd group the current set of flags and add a second check for -fcuda-rdc. I'm saying this a third time: There should be no hardcoding of versions!

Apr 9 2018, 12:56 PM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

! In D44992#1061781, @guansong wrote:
I have revert the change, let's continue to discuss how to enable bc file path better.

You are still not subscribed to openmp-commits, so again no commit email! For reference, the commit is rL329576 (and you shouldn't quote git commit hashes, there is a script in LLVM to revert SVN changes from git-svn.)

Apr 9 2018, 11:25 AM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I can see the advantage of dynamic testing. But to test everything dynamically maybe tedious. For bc file handling, I consider this is more of a project need to embed LLVM itself, using find_package is more suitable. http://llvm.org/docs/CMake.html#embedding

Apr 9 2018, 9:11 AM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I can see the advantage of dynamic testing. But to test everything dynamically maybe tedious. For bc file handling, I consider this is more of a project need to embed LLVM itself, using find_package is more suitable. http://llvm.org/docs/CMake.html#embedding

Apr 9 2018, 8:53 AM · Unknown Object (Project)
guansong committed rOMP329576: Revert "[OpenMP] enable bc file compilation using the latest clang".
Revert "[OpenMP] enable bc file compilation using the latest clang"
Apr 9 2018, 8:20 AM
guansong committed rL329576: Revert "[OpenMP] enable bc file compilation using the latest clang".
Revert "[OpenMP] enable bc file compilation using the latest clang"
Apr 9 2018, 7:49 AM

Apr 8 2018

guansong created D45415: [OpenMP] Remove extra warning when we build.
Apr 8 2018, 9:46 AM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

Currently we only enable the bc file path when we specified on the cmake command line with these flags -DLIBOMPTARGET_NVPTX_CUDA_COMPILER and -DLIBOMPTARGET_NVPTX_SELECTED_BC_LINKER.

We already enable it as soon as the user says -DLIBOMPTARGET_NVPTX_ENABLE_BCLIB and we find the Clang compiler suitable. This change breaks the build with Clang 3.9.1 and I will revert it early next week to fix this regression.

Apr 8 2018, 9:20 AM · Unknown Object (Project)
guansong added a comment to D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.

I see. I was trying to set the bc file utilities using find_pacakge(llvm...) instead of using COMPILER_ID and common it to an upper level as we search things like elf and ffi.

Apr 8 2018, 8:18 AM · Unknown Object (Project)

Apr 5 2018

guansong added a comment to D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.

That is fine. I don't think this conflicts with what you said?

Apr 5 2018, 12:57 PM · Unknown Object (Project)
guansong added a comment to D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.

This is only for compiler and linker used for building bclib. It is not the default build compiler.

Apr 5 2018, 12:37 PM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I don't think this is a regression through, as it is broken right now unless one uses an older (less 4.0) clang.

... which is enough to satisfy the definition of a regression. Yes, Clang also regressed but that's no excuse to regress the runtime library. Please revert this change or let me know if you can't and I'll do it.

I can add this flag conditionally as you suggested, how about something like this? where we check llvm version?

# Set flags for Clang cuda compilation.  Only Clang is supported because there is
# no other compiler capable of generating bitcode from cuda sources.
if (LLVM_VERSION_MAJOR GREATER 6)
  set(CUDA_FLAGS
    -emit-llvm
    -fcuda-rdc
    -O1
    -Xclang -target-feature
    -Xclang +${CUDA_PTX_VERSION}
    --cuda-device-only
    -DOMPTARGET_NVPTX_TEST=0 -DOMPTARGET_NVPTX_DEBUG=0
    )
else()
  set(CUDA_FLAGS
    -emit-llvm
    -O1
    -Xclang -target-feature
    -Xclang +${CUDA_PTX_VERSION}
    --cuda-device-only
    -DOMPTARGET_NVPTX_TEST=0 -DOMPTARGET_NVPTX_DEBUG=0
    )
endif()

No, it needs to be a compiler check. LLVM_VERSION won't be set if building openmp standalone.

Apr 5 2018, 12:29 PM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I don't think this is a regression through, as it is broken right now unless one uses an older (less 4.0) clang.

... which is enough to satisfy the definition of a regression. Yes, Clang also regressed but that's no excuse to regress the runtime library. Please revert this change or let me know if you can't and I'll do it.

Apr 5 2018, 11:48 AM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

So , will the deviceRTLs/nvptx change? Instead of extern __shared__, what will it use for those data structures?

I don't question the flag, as I said libomptarget-nvptx was the main reason to implement that functionality. What's bad about this is adding this flag unconditionally, the build system needs to properly detect if the compiler supports it.

If you can't work on that right now, I hereby request this change to be reverted as it results in a build regression.

Apr 5 2018, 7:35 AM · Unknown Object (Project)

Apr 3 2018

guansong created D45220: [OpenMP] Search llvm package to find bc file compiler and linker if not specified by a user.
Apr 3 2018, 11:38 AM · Unknown Object (Project)
guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I see. We compared the change in (4.0?) and after, realized the extern shared is turned off. We reenabled (basically allow it to pass parsing) and everything is expected.

Apr 3 2018, 11:19 AM · Unknown Object (Project)
guansong committed rL329072: [OpenMP] enable bc file compilation using the latest clang.
[OpenMP] enable bc file compilation using the latest clang
Apr 3 2018, 8:04 AM
guansong committed rOMP329072: [OpenMP] enable bc file compilation using the latest clang.
[OpenMP] enable bc file compilation using the latest clang
Apr 3 2018, 8:04 AM
guansong closed D44992: [OpenMP] enable bc file compilation using the latest clang.
Apr 3 2018, 8:04 AM · Unknown Object (Project)

Apr 2 2018

guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

My search got the same conclusion as Greg did, I don't think -fcuda-rdc will change the IR code, except allowing multiple compilation units as specified in the options.td

Apr 2 2018, 9:23 AM · Unknown Object (Project)

Apr 1 2018

guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

this fix I had is similar to 751ae2ab8, which is exactly for this.

Apr 1 2018, 10:36 AM · Unknown Object (Project)

Mar 28 2018

guansong added a comment to D44992: [OpenMP] enable bc file compilation using the latest clang.

I mean using -DLIBOMPTARGET_NVPTX_ENABLE_BCLIB=1 on cmake command line, to build the bc libraries. Did you use that flag?

Mar 28 2018, 3:27 PM · Unknown Object (Project)
guansong created D44992: [OpenMP] enable bc file compilation using the latest clang.
Mar 28 2018, 11:44 AM · Unknown Object (Project)

Jan 12 2018

guansong added inline comments to D14254: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget device RTLs..
Jan 12 2018, 9:37 AM · Unknown Object (Project)

Dec 11 2017

guansong added inline comments to D14254: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget device RTLs..
Dec 11 2017, 9:33 PM · Unknown Object (Project)

Dec 8 2017

guansong added a comment to D14254: [OpenMP] Initial implementation of OpenMP offloading library - libomptarget device RTLs..

Here is a general comment, we should follow Alex's suggestion to make a generic GPU device and extend that to NVPTX and AMDGCN. My current idea is:

Dec 8 2017, 1:32 PM · Unknown Object (Project)
guansong added a comment to D38978: [OpenMP] Enable the lowering of implicitly shared variables in OpenMP GPU-offloaded target regions to the GPU shared memory.

Here is a question, do we require that the alloca size to be compile time constant?

Dec 8 2017, 9:58 AM

Dec 7 2017

guansong added a comment to D40977: [OpenMP] NVPTX: Set default/minimum compute capability to sm_35.

LGTM.

Dec 7 2017, 12:15 PM · Unknown Object (Project)

Jul 19 2017

guansong added a project to D34784: [OpenMP] Add flag for specifying the target device architecture for OpenMP device offloading: Unknown Object (Project).
Jul 19 2017, 7:30 AM · Unknown Object (Project)

Jun 7 2017

guansong added a comment to D33227: [OpenMP] libomptarget: move debugging dumps under control of environment variable OMP_TARGET_DEBUG.

Just to share what I have locally. I have OFFLOAD_DEBUG=1 for the plugin side debug messages, and DEVICE_DEBUG=1 for the deviceRTL side messages.

So are you suggesting that each plugin has its own env variable to turn debug output on/off?

Jun 7 2017, 2:03 PM · Unknown Object (Project)
guansong added a comment to D33227: [OpenMP] libomptarget: move debugging dumps under control of environment variable OMP_TARGET_DEBUG.

Just to share what I have locally. I have OFFLOAD_DEBUG=1 for the plugin side debug messages, and DEVICE_DEBUG=1 for the deviceRTL side messages.

Jun 7 2017, 10:46 AM · Unknown Object (Project)

Mar 15 2017

guansong committed rL297896: fix build break by removing the target on command line.
fix build break by removing the target on command line
Mar 15 2017, 2:58 PM
guansong committed rL297890: enable -save-temps with -finclude-defult-header.
enable -save-temps with -finclude-defult-header
Mar 15 2017, 2:11 PM
guansong closed D30743: enable -save-temps with -finclude-defult-header by committing rL297890: enable -save-temps with -finclude-defult-header.
Mar 15 2017, 2:11 PM

Mar 14 2017

guansong updated the diff for D30743: enable -save-temps with -finclude-defult-header.

Thanks, update as suggested.

Mar 14 2017, 7:21 AM

Mar 13 2017

guansong updated the diff for D30743: enable -save-temps with -finclude-defult-header.
Mar 13 2017, 11:54 AM

Mar 10 2017

guansong updated the diff for D30784: Create a git ignore file for openmp runtime.

As suggested, mv the dir to the empty dir down below

Mar 10 2017, 12:11 PM

Mar 9 2017

guansong updated the diff for D30750: Rename PT_NOTE namespace name used in AMDGPUPTNote.h.

Use ElfNote instead of AMD_PT_NOTE as suggested.

Mar 9 2017, 12:26 PM
guansong created D30784: Create a git ignore file for openmp runtime.
Mar 9 2017, 9:33 AM

Mar 8 2017

guansong created D30750: Rename PT_NOTE namespace name used in AMDGPUPTNote.h.
Mar 8 2017, 10:53 AM
guansong added a reviewer for D30743: enable -save-temps with -finclude-defult-header: Anastasia.
Mar 8 2017, 10:13 AM
guansong updated the diff for D30743: enable -save-temps with -finclude-defult-header.

add a test

Mar 8 2017, 10:11 AM
guansong added a reviewer for D30743: enable -save-temps with -finclude-defult-header: yaxunl.
Mar 8 2017, 9:06 AM
guansong created D30743: enable -save-temps with -finclude-defult-header.
Mar 8 2017, 9:05 AM

Jul 5 2016

guansong updated subscribers of D21840: [Driver][CUDA][OpenMP] Reimplement tool selection in the driver..
Jul 5 2016, 10:45 AM
guansong updated subscribers of D18172: [CUDA][OpenMP] Add a generic offload action builder.
Jul 5 2016, 10:44 AM
guansong updated subscribers of D18171: [CUDA][OpenMP] Create generic offload action.
Jul 5 2016, 10:44 AM

Jun 14 2016

guansong updated subscribers of D20682: Expose target-specific pre-linking passes in TargetMachine.
Jun 14 2016, 11:45 AM

Jun 8 2016

guansong updated subscribers of D13909: clang-offload-bundler - offload files bundling/unbundling tool.
Jun 8 2016, 9:33 AM
guansong updated subscribers of D18170: [CUDA][OpenMP] Create generic offload toolchains.
Jun 8 2016, 7:08 AM

Apr 6 2016

guansong updated subscribers of D18110: [OpenMP] Fix SEMA bug in the capture of global variables in template functions..
Apr 6 2016, 2:13 PM

Feb 17 2016

guansong added a comment to D17323: [OpenMP]Update to clang 3.8 ReleaseNotes.rst.

Should we mention that the runtime can be built "in-tree" as well? This simplifies users experience to access OpenMP support.

Feb 17 2016, 8:28 AM
guansong updated subscribers of D17323: [OpenMP]Update to clang 3.8 ReleaseNotes.rst.
Feb 17 2016, 8:15 AM