MyDeveloperDay added a comment.

In D92753#2436852 <https://reviews.llvm.org/D92753#2436852>, @curdeius wrote:

> You've added `messUp` parameter to `verifyFormat`, because, IIUC, pragmas 
> wouldn't be at the desired scope level indentation otherwise.

messUp was pull the line below onto the pragma line, which was then unable to 
determine when if it should break it

i.e.

  #pragma omp simd
  for (int k = 0; k < 10; k++) {

became

  #pragma omp simd for (int k = 0; k < 10; k++) {

I'm not sure why this occurred, but I assume there is no real token parsing of 
an #pragam line.


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

https://reviews.llvm.org/D92753

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

Reply via email to