Replace Accel-Sim and GPGPU-Sim build system from Make to CMake

William-An commented 9 months ago

The current build system for Accel-Sim and GPGPU-Sim is based on Make, whose output and content are too messy and hard to read. Another issue with the current build system is that extra precautions are needed when adding new modules to the simulator. For instance, several variables like SIM_OBJ_FILES_DIR and compiler flags need to be manually passed down the makefiles in each subfolder. By changing to CMake, the weird build dependency bug in #115 can also be addressed.

Switching to CMake will enable the following features:

Checks for package dependency (find_package) and abort config/build if necessary
Checks for environment variables and abort config/build if necessary
Cleaner compilation messages and build progress bar support, a sample output looks like the following:
Automated dependency management
Simpler CMakeLists.txt compared with Makefile
Switch to different build backends like ninja or Visual studio if needed
Simpler and intuitive build options passing (easily build for release and debug without the need to run setup script)
Integrate setup script as part of build configuration

JRPan commented 9 months ago

Did you implement this already?

William-An commented 9 months ago

Nope, but planning to get it to work.

JRPan commented 7 months ago

Missing:

[ ] OpenCL support
[ ] cuobjdump_to_ptxplus

William-An commented 1 month ago

For this to mark complete, what are the remaining tasks? Besides documentation and the above two components compilation?

JRPan commented 1 month ago

well I guess you can just write docs and say:

if you want opencl and cuobjdump_to_ptxplus just use make

and write up how to use cmake.

tgrogers commented 1 month ago

Yeah, at this point, I don't know if anyone cares about OpenCL 1.0 or the SASS 1.3 ptxplus emulation. It's not worth maintaining the path for new code, just mention you can use make if you want to use these older modes.

On Thu, Oct 3, 2024 at 6:59 PM JRPan @.***> wrote:

well I guess you can just write docs and say:

if you want opencl and cuobjdump_to_ptxplus just use make

and write up how to use cmake.

— Reply to this email directly, view it on GitHub https://github.com/accel-sim/accel-sim-framework/issues/279#issuecomment-2392478732, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7UY4IJAHONKEX6X2UW6QTZZXD33AVCNFSM6AAAAABC4HSXC6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJSGQ3TQNZTGI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

accel-sim / accel-sim-framework

Replace Accel-Sim and GPGPU-Sim build system from Make to CMake #279