[PATCH] D23279: clang-reorder-fields

Alexander Shaposhnikov via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 14:42:08 PDT 2016


alexshap added inline comments.

================
Comment at: test/clang-reorder-fields/CStructAmbiguousName.cpp:6
@@ +5,3 @@
+struct Foo {
+  int x;    // CHECK: int x;
+  double y; // CHECK: double y;
----------------
djasper wrote:
> Have you thought about how to handle comments that surround these?
yup, take a look at one of my comments above.
Basically my plan was to leverage the approach/api described here
http://llvm.org/devmtg/2012-11/Gribenko_CommentParsing.pdf
It's doable, not very complicated, but requires some work - i need time to think more about it and prepare a diff - it's still in the oven and not ready yet. 


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list