<div dir="ltr">I applied the patch on top of the clang13 RC and It seems not to be generating any replacements. I haven't had the time to dig and understand why. Diagnostics seem to be correct.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Aug 15, 2021 at 12:16 PM Jonas Toth via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">JonasToth added a comment.<br>
<br>
The check in the latest form can properly transform `llvm/lib` and `clang/` and requires 28 manual fixes afterwards (values and references are transformed, no other modifications).<br>
 Some are good and not an issue with the analysis itself, some may or may not be an issue with the analysis itself. I think it is more a template meta programming thing.<br>
<br>
Here is the branch that contains the changes: <a href="https://github.com/JonasToth/llvm-project/tree/transformation/2021-august-15-revision2" rel="noreferrer" target="_blank">https://github.com/JonasToth/llvm-project/tree/transformation/2021-august-15-revision2</a><br>
<br>
Short Stats:<br>
<br>
- llvm/lib transformation: `1724 files changed, 62110 insertions(+), 62110 deletions`; `26 manual fixes`<br>
- clang/ transformation: `819 files changed, 24166 insertions(+), 24166 deletions`; `2 manual fixes`, one fix solves a ternary-operator issue where only one branch got `const`, the other one adds default construction<br>
<br>
I consider all 27 of them as false positives, but they might be bad const-correctness of the library code as well.<br>
Thats good enough?<br>
<br>
@0x8000-0000 @tiagoma If you have time it would be great if you could check the transformation on your code bases.<br>
Everyone else is welcome to try it out too!<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D54943/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54943/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D54943" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54943</a><br>
<br>
</blockquote></div>