<div dir="ltr"><div>My usual take on this would be that it's within the LLVM project norms to fix up naming on a case by case basis (independent of the recent discussion you mentioned) - especially if different subsets of a single interface/group of related interfaces have become more visibly inconsistent.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 17, 2020 at 4:04 AM Ties Stuij 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">Hi there,<br>
<br>
At the end of last week we saw a number of commits go in that were camelCasing batches of MCStreamer::Emit* and AsmPrinter::Emit* functions.<br>
<br>
For example:<br>
- <a href="https://reviews.llvm.org/rG549b436beb4129854e729a3e1398f03429149691" rel="noreferrer" target="_blank">https://reviews.llvm.org/rG549b436beb4129854e729a3e1398f03429149691</a><br>
- <a href="https://reviews.llvm.org/rGa55daa146166353236aa528546397226bee9363b" rel="noreferrer" target="_blank">https://reviews.llvm.org/rGa55daa146166353236aa528546397226bee9363b</a><br>
- <a href="https://reviews.llvm.org/rG0bc77a0f0d1606520c7ad0ea72c434661786a956" rel="noreferrer" target="_blank">https://reviews.llvm.org/rG0bc77a0f0d1606520c7ad0ea72c434661786a956</a><br>
<br>
Unfortunately all these individual commits trigger the same merge conflicts over and over again with our downstream repo, which takes us some manual intervention every time.<br>
<br>
I understand uniformity is a nice to have, but:<br>
1 - is it worth it to do this work right now? I can remember the casing debate a few months back, which seems unrelated to this work which seems manual, but I'm unsure of the outcome.<br>
2 - If this work should be done, it would be nice if all of the work is done in one batch, to save us some of the downstream overhead.<br>
<br>
Thanks<br>
/Ties<br>
_______________________________________________<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>