[PATCH] D16940: Fix build LLVM with -D LLVM_USE_INTEL_JITEVENTS:BOOL=ON on Windows

[Andy Kaylor via llvm-commits]

andrew.w.kaylor added inline comments.

================
Comment at: lib/ExecutionEngine/IntelJITEvents/CMakeLists.txt:18
@@ +17,2 @@
+
+add_dependencies(LLVMIntelJITEvents intrinsics_gen)
----------------
rafael wrote:
> Why do you need this?
That's a good question.  There is no reason that IntelJITEvents should need this, but there is an include of attributes.inc a few levels deep in a file the gets included from IntelJITEventListener.cpp.  Something has changed in trunk in the past few days to make the build failure go away, but I haven't been able to track down what it was.  The build failure still exists in the 3.8 branch.  It looks like this:
```
Building CXX object lib/ExecutionEngine/IntelJITEvents/CMakeFiles/LLVMIntelJITEvents.dir/IntelJITEventListener.cpp.o
In file included from /users/akaylor/llvm38/llvm/include/llvm/IR/CallSite.h:31: ,
                 from /users/akaylor/llvm38/llvm/include/llvm/Analysis/AliasAnalysis.h:42,
                 from /users/akaylor/llvm38/llvm/include/llvm/CodeGen/MachineInstr.h:26,
                 from /users/akaylor/llvm38/llvm/include/llvm/CodeGen/MachineBasicBlock.h:18,
                 from /users/akaylor/llvm38/llvm/include/llvm/CodeGen/MachineFunction.h:22,
                 from /users/akaylor/llvm38/llvm/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp:18:
/users/akaylor/llvm38/llvm/include/llvm/IR/Attributes.h:69:38: fatal error: llvm/IR/Attributes.inc: No such file or directory
     #include "llvm/IR/Attributes.inc"
                                      ^
compilation terminated.
```
It may be that having the dependency here isn't the best way to fix this.


Repository:
  rL LLVM

http://reviews.llvm.org/D16940