[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