MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

I ran the examples from https://bugs.llvm.org/show_bug.cgi?id=38314 and its so 
much better than the previous state!

Thanks for this patch, LGTM

  namespace n {
  void foo() {
      {
          {
              {
                  statement();
                  if (false) {
                  }
              }
          }
          {}
      }
  }
  }  // namespace n
  
  void foo() {
      {
          {
              statement();
              if (false) {
              }
          }
      }
      {}
  }  // namespace n


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

https://reviews.llvm.org/D114583

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

Reply via email to