<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 18, 2021 at 4:07 AM Luke Drummond <<a href="mailto:luke.drummond@codeplay.com">luke.drummond@codeplay.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu Jun 17, 2021 at 7:15 PM BST, Mehdi AMINI wrote:<br>
> On Thu, Jun 17, 2021 at 10:15 AM David Blaikie via llvm-dev <<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> > Got links to the reviews? Hard to discuss in the abstract.<br>
> ><br>
> > But generally, if it is intended to change observable behavior of the LLVM<br>
> > program (if you have to modify a lit test, that's not NFC, if one could<br>
> > craft a test (that doesn't invoke UB, though that shouldn't be possible<br>
> > through the command line interface, etc) that would need to be modified<br>
> > when the patch is committed - then it's not NFC).<br>
> ><br>
><br>
> That's my litmus test: I see NFC is an indication that no test changes and<br>
> none are expected to be provided. The functional behavior of the compiler is<br>
> unchanged. I use NFC on API changes and refactoring when it fits this<br>
> description.<br>
<br>
I think this is a bit liberal as it ignores API users - unless I'm<br>
misunderstanding your statement about what you mean by "API changes".</blockquote><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:0 0 0 .8ex;border-left:1px #ccc solid;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>