Anastasia added a comment.

In D96515#2578629 <https://reviews.llvm.org/D96515#2578629>, @thakis wrote:

> Looks like this breaks tests: http://45.33.8.238/linux/39988/step_8.txt
>
> Please take a look, and revert for now if it takes a while to fix.

Thanks for pointing this out. I think the following change should fix the 
issue, but I am still trying to reproduce it locally as it is taking time to 
build.

  diff --git a/clang-tools-extra/test/pp-trace/pp-trace-pragma-opencl.cpp 
b/clang-tools-extra/test/pp-trace/pp-trace-pragma-opencl.cpp
  index 8ba26c3b7396..31f61027994f 100644
  --- a/clang-tools-extra/test/pp-trace/pp-trace-pragma-opencl.cpp
  +++ b/clang-tools-extra/test/pp-trace/pp-trace-pragma-opencl.cpp
  @@ -1,4 +1,4 @@
  -// RUN: pp-trace -callbacks '*,-FileChanged,-MacroDefined' %s -- -x cl | 
FileCheck --strict-whitespace %s
  +// RUN: pp-trace -callbacks '*,-FileChanged,-MacroDefined' %s -- -x cl 
-cl-no-stdinc | FileCheck --strict-whitespace %s
   
   #pragma OPENCL EXTENSION all : disable
   #pragma OPENCL EXTENSION cl_khr_int64_base_atomics : disable

FYI, @thakis would you be able to approve the fix? Otherwise, I am also ok to 
revert it until I figure out how to commit the fix.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96515/new/

https://reviews.llvm.org/D96515

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to