RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-06 Thread Samuel F Antao via cfe-commits
Date: 01/05/2016 07:58 PM Subject: RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata. Hi Samuel The issue is not related to capture feature but the order of evaluation which is not specified for arguments in a function call in C/C++. Wit

RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-05 Thread Liao, Michael via cfe-commits
: Tuesday, January 5, 2016 4:25 PM To: Liao, Michael Cc: cfe-commits ; Artem Belevich Subject: RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata. Hi Michael, Thanks for the patch! I am not sure I understand why the behavior is unspecified if Idx is captured by

RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-05 Thread Samuel F Antao via cfe-commits
fe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf Of Samuel F Antao via cfe-commits Sent: Tuesday, January 5, 2016 10:57 AM To: Artem Belevich Cc: cfe-commits Subject: Re: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata. Hi Art, That only fixed one of th

RE: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-05 Thread Liao, Michael via cfe-commits
...@lists.llvm.org] On Behalf Of Samuel F Antao via cfe-commits Sent: Tuesday, January 5, 2016 10:57 AM To: Artem Belevich Cc: cfe-commits Subject: Re: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata. Hi Art, That only fixed one of the problems. The other one I am having some

Re: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-05 Thread Samuel F Antao via cfe-commits
01:54 PM Subject:Re: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata. Samuel, The tests are still failing: http://lab.llvm.org:8011/builders/clang-bpf-build/builds/5759 On Tue, Jan 5, 2016 at 10:02 AM, Samuel Antao via cfe-commits < cfe-comm

Re: r256854 - [OpenMP] Allow file ID to be signed in the offloading metadata.

2016-01-05 Thread Artem Belevich via cfe-commits
Samuel, The tests are still failing: http://lab.llvm.org:8011/builders/clang-bpf-build/builds/5759 On Tue, Jan 5, 2016 at 10:02 AM, Samuel Antao via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: sfantao > Date: Tue Jan 5 12:02:24 2016 > New Revision: 256854 > > URL: http://llvm.o