Jonathan Smith
Jonathan Smith
If you're fine with moving to Visual Studio 2022, I can handle this.
> Hi @banach-space @jvstech. Thank you for your work on this tutor. > > [Enzyme](https://github.com/EnzymeAD/Enzyme) is an established open source tool gaining prominence in the scientific computing community. It is...
I've had some more time to work on this, and I've run into an issue that's been discussed before elsewhere. Each pass is its own library. The transforms need to...
I won't be much help until maybe tomorrow, but after a very cursory glance, you'll have to link against `LLVMSupport.lib` (which should be handled automatically by LLVM's CMake modules, and...
Okay, I think I see what's happening. I'm still building LLVM at the moment, but after taking a closer look at `enzyme/Enzyme/CMakeLists.txt`: https://github.com/EnzymeAD/Enzyme/blob/11cc0f13264671094bf41ff280ba2086f47d2f40/enzyme/Enzyme/CMakeLists.txt#L53-L82 If you're linking against `LLVM.dll` (which is...
@OwenTrokeBillard Not a problem. Happy to help in any way I can. I wasn't sure if `LLVM.dll` was a thing, myself. I knew `LLVM-C.dll` could be built, but obviously, that...
By the way, you may be interested in this [draft pull request with a prototype for building LLVM as DLLs on Windows](https://github.com/llvm/llvm-project/pull/67502). It's not done yet, but there is _effort_...
AH HA! I figured it out! https://github.com/EnzymeAD/Enzyme/blob/b6bb8d85b1b51bb4d531937d1f4a80f174b5a4e4/enzyme/Enzyme/Enzyme.cpp#L28-L36 And, specifically: https://github.com/EnzymeAD/Enzyme/blob/b6bb8d85b1b51bb4d531937d1f4a80f174b5a4e4/enzyme/Enzyme/Enzyme.cpp#L29 **Do not do this.** It's forbidden by the C++ standard because those access specifiers are part of the ABI. I...
> Looks like the problem is that the exit limit type in scalar evolution is marked private in LLVM but used by us. This should hopefully be a simple patch...
> @jvstech Thank you for thoroughly investigating this. I never would have suspected redefining access specifiers was the culprit. You have saved me weeks of toiling. Not a problem. It's...