search

dtkerns / d2

OSS implementation of DOSAGE
MIT License
1 stars 0 forks source link

Specify LLVM versions in installation and usage instruction #6

Open abhishektiwari opened 3 weeks ago

abhishektiwari commented 3 weeks ago

Got following warning during installation, please specify supported LLVM versions in installation and usage instruction to ensure users can follow the instructions when using LLVM versions < 19 or 19>=.

warning: due to a bug in LLVM versions < 19, you will need to add a -O1 (minimum) to CFLAGS in your applications makefiles

For JOSS Review

dtkerns commented 3 weeks ago

That's correct, please see issue #3. All versions of LLVM are supported. However, the original work was done under CentOS7 which was/is running LLVM version 3.4.2. (which still runs without issue) While "modernizing" to RHEL9 (which runs LLVM version 16.0.6) the loop bug was discovered and documented. Due to the nature/philosophy of RHEL, it is not likely RHEL 9 will ever go beyond the the 16 series. The LLVM bug was fixed in version 19. Thus, users running LLVM versions > 13 and < 19, will need to add the -O1 (minimum optimization level) in order to successfully get the loop output from the LLVM opt command.