Eugene.Zelenko added a comment.

In D54943#1815912 <https://reviews.llvm.org/D54943#1815912>, @0x8000-0000 wrote:

> As an aside, once this is merged in, I dream of a "fix-it" for old style C 
> code:
>
>   int foo;
>  
>   ...
>   /* page of code which does not use either foo or bar */
>   ...
>   foo = 5;
>
>
> Moves the foo declaration to the line where it is actually first initialized. 
> Then run the constify tool on top.


You are not first :-)  See PR21983.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D54943



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

Reply via email to