[Mlir-commits] [mlir] [mlir][XeVM] Use libocloc API for binary generation. (PR #188353)

[Md Abdullah Shahneous Bari]

================
@@ -39,16 +39,102 @@
 #include "llvm/Support/raw_ostream.h"
 #include "llvm/Target/TargetMachine.h"
 
-#if MLIR_XEVM_OCLOC_AVAILABLE
-#include <ocloc_api.h>
-#endif // MLIR_XEVM_OCLOC_AVAILABLE
-
 #include <cstdint>
 #include <cstdlib>
 
 using namespace mlir;
 using namespace mlir::xevm;
 
+#if MLIR_XEVM_OCLOC_LIB_AVAILABLE
----------------
mshahneo wrote:

Actually, I intentionally didn't put it in the header (the chance of being the header included in other files). Since, we are only using the libocloc APIs in this file, wanted to keep the external declaration contained in this translation unit.  Do you think we should still put it in the header?

https://github.com/llvm/llvm-project/pull/188353