<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 14, 2022 at 4:29 PM Nikita Popov <<a href="mailto:nikita.ppv@gmail.com">nikita.ppv@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Feb 13, 2022 at 12:37 PM Florian Hahn via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
Author: Florian Hahn<br>
Date: 2022-02-13T11:36:54Z<br>
New Revision: d462e647542dc54718a4a4dfa6476b3dd2649a13<br>
<br>
URL: <a href="https://github.com/llvm/llvm-project/commit/d462e647542dc54718a4a4dfa6476b3dd2649a13" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d462e647542dc54718a4a4dfa6476b3dd2649a13</a><br>
DIFF: <a href="https://github.com/llvm/llvm-project/commit/d462e647542dc54718a4a4dfa6476b3dd2649a13.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d462e647542dc54718a4a4dfa6476b3dd2649a13.diff</a><br>
<br>
LOG: [LV] Drop noalias from check lines from test (NFC).<br>
<br>
The noalias metadata checks re not really relevant for the test and<br>
slight changes to metadata numbering can have large knock-on effects<br>
causing large noise in test diff.<br></blockquote><div><br></div><div>Manually dropping the metadata causes even more noise to appear the next time someone runs update_test_checks on this file. Please avoid manually editing UTC-generated files in this way. If metadata does not add value to this test, then I'd recommend adding a new --scrub-metadata or similar flag to UTC (to complement --scrub-attributes), which allows dropping irrelevant information while preserving the automation.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"></blockquote></div></div></blockquote><div><br></div><div> </div><div>Thanks, I'll use the flag nex time.</div><div><br></div><div>FWIW auto-generating check lines for tests like this one isn't great in the first place IMO, because small changes to the output cause a huge amount of noise. It's easier to update, but makes it much harder to review changes :(</div><div><br></div><div>Is there a flag for update_test_checks.py to avoid regenerating check lines if only value numbering changes in a sub-region?</div></div></div>