[PATCH] D23279: clang-reorder-fields

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 10:50:59 PDT 2016


omtcyfz added a comment.

In https://reviews.llvm.org/D23279#510147, @alexshap wrote:

> Suppose you have a struct Foo defined in LibA and you have used clang-rename to rename Foo into Bar (or just a field of Foo say - you have renamed Foo::x into Foo::y); LibA is linked to LibB and you have Foo foo = { &x, 17, 1.29, 0 }; in LibB. Therefore LibA knows nothing of struct Foo usages in LibB. Thus, the struct constructions and everything related stays exactly the same in LibB.
>
> It's not clear to me why the issue is specific to this particular tool (maybe i am bit sleepy and missing smth).


Because `LibB` becomes broken.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list