<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>For a while now I have noticed that when I build LLVM using CMake on my OS X machine there are two dynamically linked libraries, BugpointPasses and LLVMHello, that are built with the extension ".so" instead of the extension ".dylib". This has been causing four test cases to fail when running "make check".</div><div><br></div><div>The attached patch modifies add_llvm_loadable_module in the CMake build system with a piece of logic found in add_clang_library in Clang's build system. I'm not quite sure what the logic is doing, but it did cause the two preview libraries to be emitted with the correct ".dylib" extension and allowed "make check" to run successfully.</div><div><br></div><div>I've tested this with the two systems that I regularly build LLVM on:</div><div><span class="Apple-tab-span" style="white-space:pre">  </span>1. Console build, 64-bit OS X but emitting 32-bit libraries</div><div><span class="Apple-tab-span" style="white-space:pre">  </span>2. Console build, 64-bit Linux</div><div><br></div><div>The patch has not been tested using XCode or any other build target.</div><div><br></div><div><div>
<div><span class="Apple-style-span" style="font-family: Helvetica; ">--</span></div><div><div style="font-family: Helvetica; ">Wesley Peck</div><div style="font-family: Helvetica; ">University of Kansas</div><div style="font-family: Helvetica; ">SLDG Laboratory</div></div>
</div>
<br></div><div></div></body></html>