[PATCH] D52938: [CUDA] Use all 64 bits of GUID in __nv_module_id

This revision was automatically updated to reflect the committed changes. Closed by commit rL343875: [CUDA] Use all 64 bits of GUID in __nv_module_id (authored by tra, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D52938?vs=168489&id=

[PATCH] D52938: [CUDA] Use all 64 bits of GUID in __nv_module_id

tra added a comment. This particular change is largely cosmetic. I've just spotted this nit while I was debugging a different problem. It's also related to module ID. We're trying to compile NCCL 2.3 with -fcuda-rdc and we were getting duplicate symbols when we tried to link multiple object fi

[PATCH] D52938: [CUDA] Use all 64 bits of GUID in __nv_module_id

Hahnfeld accepted this revision. Hahnfeld added a comment. This revision is now accepted and ready to land. LG. Out of interest: Is this fixing a particular issue? https://reviews.llvm.org/D52938 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D52938: [CUDA] Use all 64 bits of GUID in __nv_module_id

tra created this revision. tra added a reviewer: Hahnfeld. Herald added subscribers: bixia, jlebar, sanjoy. getGUID() returns an uint64_t and "%x" only prints 32 bits of it. Use PRIx64 format string to print all 64 bits. https://reviews.llvm.org/D52938 Files: clang/lib/CodeGen/CGCUDANV.cpp