[Kaleidoscope] Fix Kaleidoscope JIT symbol resolution error on Windows

[justice_adams]

Summary

Currently, when a user builds and runs the Kaleidoscope tutorial on Windows they will run into the following error when attempting to execute basic statements through the JIT

Assertion failed: (KV.second.getFlags() & ~WeakFlags) == (I->second & ~WeakFlags) && "Resolving symbol with incorrect flags", file llvm\lib\ExecutionEngine\Orc\Core.cpp, line 2674

This patch fixes the issue by claiming responsibility for COFF symbols when the user is running on Windows. This is exactly how LLJIT does it, so I mimicked it here.
https://reviews.llvm.org/source/llvm-github/browse/main/llvm/lib/ExecutionEngine/Orc/LLJIT.cpp$669

[lhames]

LGTM. Thanks Justice!

[justice_adams]

Hi @lhames , thanks for the review. Could I have you commit this for me? I do not have commit access

[justice_adams]

@lhames Ping to commit this if you're able :)

[lhames]

Sorry for the very belated update on this: It looks like the fix was committed in da83b70a6fe for https://reviews.llvm.org/D108348 (which was actually submitted after this was accepted, my apologies for the mix up!).

llvm/examples/Kaleidoscope/include/KaleidoscopeJIT.h

KaleidoscopeJIT(std::unique_ptr<ExecutorProcessControl> EPC,
         ObjectLayer(*this->ES,
                     []() { return std::make_unique<SectionMemoryManager>(); }),
         CompileLayer(*this->ES, ObjectLayer,
                      std::make_unique<ConcurrentIRCompiler>(std::move(JTMB))),
         MainJD(this->ES->createBareJITDylib("<main>")) {
     MainJD.addGenerator(
         cantFail(DynamicLibrarySearchGenerator::GetForCurrentProcess(
             DL.getGlobalPrefix())));
+
+    if (JTMB.getTargetTriple().isOSBinFormatCOFF()) {
+      ObjectLayer.setOverrideObjectFlagsWithResponsibilityFlags(true);
+      ObjectLayer.setAutoClaimResponsibilityForObjectSymbols(true);
+    }
   }
 
   ~KaleidoscopeJIT() {
     if (auto Err = ES->endSession())
       ES->reportError(std::move(Err));
   }
 
   static Expected<std::unique_ptr<KaleidoscopeJIT>> Create() {