Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TODO list, Help wanted #3

Open
Dead2 opened this issue Sep 16, 2022 · 1 comment
Open

TODO list, Help wanted #3

Dead2 opened this issue Sep 16, 2022 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Dead2
Copy link
Owner

Dead2 commented Sep 16, 2022

This is a list of suggested improvements that anyone can contribute to, I will try to keep it updated as we add/remove items.
Please try not to clutter this thread with replies related to the below. If need be you can open a new issue dedicated to discussing one of the points below.

Wanted improvements:

  • Fix for SZ_STACK related crashes.
  • Fix for SZ_CODE related crashes.
  • Port to ARM/Aarch64.
  • Port to S390.
  • Port to Windows.
  • More and better comments, especially useful in llvm-pass code that may be hard to follow and debug for beginners.
  • Investigate whether pass/IntrinsicLibcalls.h needs additional Intrinsics added due to new ones having been added in later llvm versions. [1]
  • Improved tests [2].
  • Github Actions CI testing, preferably also running the tests on all supported platforms and architectures.

1: There is very little info about why these intrinsics are on the list and others are not, if you know the details then please create a PR with
improved comments to document this feature.

2: The current tests are not able to detect the Stack and Code crashes, and it is hard to determine whether the current tests actually passed or failed due to an abundance of hard-to-understand output.
An option to improve test understanding would be to change the project from hand-written makefiles to CMake and then utilize CTest.

@Dead2 Dead2 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Sep 16, 2022
@magras
Copy link
Collaborator

magras commented Nov 26, 2022

I'd like to add a couple tasks to the todo list:

  • Rewrite stabilizer for the new llvm pass manager because legacy pass manager is deprecated.
  • Porting to llvm 15 may require breaking compatibility with previous versions of llvm because clang 15 finally started to use opaque pointers. It might be possible to keep backward compatibility with older versions of llvm, but I have no idea how hard that would be.

I think it makes sense to do both tasks in one go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants