<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 6, 2019 at 10:43 PM Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.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"><div dir="ltr">Instead of bumping up the threshold, can we detect a cycle in reference files? We have `llvm::sys::fs::equivalent` to determine whether two file descriptors are for the same file or not. It doesn't seem too hard to detect a cycle using that function.</div></blockquote><div><br></div><div>This was my initial thought. In my opinion, there should be no artificial limit, but I can also see a reasonable argument to have such a limit, so I don't really mind either way.</div><div><br></div><div>-- chris</div></div></div>