[PATCH] D73335: clang-format: [JS] options for arrow functions.

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 27 08:45:03 PST 2020


RKSimon added inline comments.


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3165
+      }
+    }
+
----------------
@mprobst - this is breaking buildbots: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/27196/steps/bootstrap%20clang/logs/stdio

Please can you remove the default case and add a suitable llvm_unreachable() after the switch statement ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73335





More information about the cfe-commits mailing list