<div dir="ltr">I use modules for all my builds, so that caused them to break.<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 14, 2018 at 4:58 PM Daniel Sanders <<a href="mailto:daniel_l_sanders@apple.com">daniel_l_sanders@apple.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">Thanks. I just got a bug report about this from an out of tree target and went to fix it but you beat me to it.<br>
<br>
Out of curiosity, was it one of the LLVM bots that noticed this? I haven't seen it in any of the mail I got from the bots.<br>
<br>
> On 14 Dec 2018, at 21:04, Eric Fiselier via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> <br>
> Author: ericwf<br>
> Date: Fri Dec 14 13:04:00 2018<br>
> New Revision: 349193<br>
> <br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=349193&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=349193&view=rev</a><br>
> Log:<br>
> Add missing includes and forward decls to unbreak build<br>
> <br>
> Modified:<br>
>    llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h<br>
> <br>
> Modified: llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h?rev=349193&r1=349192&r2=349193&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h?rev=349193&r1=349192&r2=349193&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h (original)<br>
> +++ llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelWorkList.h Fri Dec 14 13:04:00 2018<br>
> @@ -12,11 +12,13 @@<br>
> <br>
> #include "llvm/ADT/DenseMap.h"<br>
> #include "llvm/ADT/SmallVector.h"<br>
> +#include "llvm/CodeGen/MachineBasicBlock.h"<br>
> +#include "llvm/CodeGen/MachineInstr.h"<br>
> #include "llvm/Support/Debug.h"<br>
> <br>
> namespace llvm {<br>
> <br>
> -class MachineInstr;<br>
> +class MachineFunction;<br>
> <br>
> // Worklist which mostly works similar to InstCombineWorkList, but on<br>
> // MachineInstrs. The main difference with something like a SetVector is that<br>
> <br>
> <br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div>