erichkeane added inline comments.
================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:1946 + FunctionEmissionStatus FES = getEmissionStatus(FD); + switch (FES) { + case FunctionEmissionStatus::Emitted: ---------------- pmrao wrote: > erichkeane wrote: > > Tabs here instead of spaces. > > > > I think this is the right fix for SYCL, but @bader should double-check > > that he's ok with this or OMP. > > Tabs here instead of spaces. > > Are you suggesting that I change the spaces to tabs instead? The reverse. Change the tabs you have at the beginning of the line to spaces. I believe that is what the double-arrow character in Phab represents (a tab). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97573/new/ https://reviews.llvm.org/D97573 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits