<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">On Thu, Nov 8, 2018 at 2:53 PM Cameron McInally <<a href="mailto:cameron.mcinally@nyu.edu">cameron.mcinally@nyu.edu</a>> wrote:<br><div class="gmail_quote"><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 dir="ltr">Hey list,<div><br></div><div>I've made a small change to some enum values in include/llvm/IR/Instruction.def and it seems to be breaking a fragile test of the MergeFunctions pass. This seemingly small change is exaggerated by the hashing function used by MergeFunctions. Since the hashes are used to sort internal MergeFunctions data structures, the output of this pass is different enough to break the test.</div></div></div></blockquote><div><br></div><div>A little digging found that the test in question, namely test/Transforms/MergeFunc/call-and-invoke-with-ranges.ll, was modified the last time the enum values in include/llvm/IR/Instruction.def were changed. This was revision <span style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px">r255522.</span></div><div><br></div><div><font color="#000000" face="Helvetica"><span style="font-size:12px">It seems that it would be safe to make a similar change </span></font><span style="color:rgb(0,0,0);font-family:Helvetica;font-size:12px">for my work on </span>D53877. Are there any objections to this?</div></div></div></div></div></div></div></div></div></div>