<div dir="ltr"><div dir="ltr"><div>Outsider comment here: I would consider an API change as NFC only if the change causes code relying on the previous version of the API to fail compile. At that point it is obvious that the API has changed and a fix is needed, although it may not be obvious what that fix needs to be.</div><div><br></div><div>On some occasions I have dealt with API changes where compiling my old code with the new API results in a correctly-compiling program. However, the resulting application fails to run correctly. This issue is much harder to track down.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 18, 2021 at 9:26 AM Mehdi AMINI via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.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"><br><div><div class="gmail_quote"><div dir="auto"><br></div><div dir="auto">Yes I am ignoring API users, I am on the same line as Nikita here.</div><div dir="auto">We don’t have stable APIs (other than the C one), so I for example I may change an API that was taking 3 bools to take now a struct parameter wrapping the 3 bools instead. I’ll tag it NFC.</div><div dir="auto"><br></div><div dir="auto">On the same line as my comment above, if I review a patch without any tests, I will ask if it NFC.</div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto"><br></div><div dir="auto">— </div><div dir="auto">Mehdi </div><div dir="auto"><br></div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" dir="auto"><br>
<br>
> We could improve the doc maybe?<br>
I'm happy to do this legwork but will hold off until something of a consensus<br>
emerges.<br>
<br>
All the Best<br>
<br>
Luke<br>
-- <br>
Codeplay Software Ltd.<br>
Company registered in England and Wales, number: 04567874<br>
Registered office: Regent House, 316 Beulah Hill, London, SE19 3HF<br>
</blockquote></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>