<div dir="ltr"><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"><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" target="_blank">luke.drummond@codeplay.com</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">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></div></blockquote><div><br></div><div>+1<br></div></div></div>