search

microsoft / llvm-mctoll

llvm-mctoll
Other
816 stars 125 forks source link

support shared libs configuration #175

Closed sv99 closed 2 years ago

sv99 commented 2 years ago

Extracted common raiser files in the Raiser library and support for shared library configuration.

bharadwajy commented 2 years ago

Thanks for your interest in the project and the PR. I'm interested in learning more about motivation for and intended use of this change?

Is the intent to encapsulate the raising functionality of llvm-mctoll as a shared library that can in turn be used in other (LLVM-based) tools?

sv99 commented 2 years ago

I plan to use it for raising single function in the IDA Pro decompiller or in the Ghidra.

bharadwajy commented 2 years ago

I plan to use it for raising single function in the IDA Pro decompiller or in the Ghidra.

Can you point to the shared library built as a result of these changes that you plan to use for this purpose?

sv99 commented 2 years ago

I build LLVM with -DBUILD_SHARED_LIBS=ON for reduce used disk space. Now the method of use has not yet been definitively determined. I am exploring different variants.

bharadwajy commented 2 years ago

Can you change the various CMakeLists.txt files to prefix all libraries built from this repo with mctoll? For e.g., the library libRaiser.so currently generated would be renamed as libmctollRaiser.so, libX66Raiser.so would be renamed libmctollX86Raiser.so etc. This will be in keeping with the apparent convention followed during creation of LLVM and clang libraries.

sv99 commented 2 years ago

libs renamed

bharadwajy commented 2 years ago

@sv99 Please sign your commits to ensure the changes get merged. Thx!

sv99 commented 2 years ago

What I need to do?

bharadwajy commented 2 years ago

What I need to do?

Look at information in the section "Merging is blocked" of the current PR and the link provided therein and follow the suggested procedure.

sv99 commented 2 years ago

sign this...