I don't think this is caused by this patch, but I'm unable to build this using clang trunk (I'm building openmp as a runtime).
/work/llvm-src/openmp/libomptarget/plugins/remote/server/Server.cpp:169:9: error: non-const lvalue reference to type 'AsyncInfoTy' cannot bind to a temporary of type '__tgt_async_info *' (__tgt_async_info *)AsyncInfo)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /work/llvm-src/openmp/libomptarget/src/device.h:223:36: note: passing argument to parameter 'AsyncInfo' here int32_t synchronize(AsyncInfoTy &AsyncInfo); ^
Has anyone else run into this? What version of clang (or gcc?) do you commonly use to build this code? Thanks!