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

Using ar/link wrapper to check inclusion correctness. #1040

Open
chen3feng opened this issue Apr 22, 2024 · 0 comments
Open

Using ar/link wrapper to check inclusion correctness. #1040

chen3feng opened this issue Apr 22, 2024 · 0 comments

Comments

@chen3feng
Copy link
Owner

chen3feng commented Apr 22, 2024

Currently, blade use a <target-name>.incchk file to pass infomation to the inclusion check script, it generates a <target-name>.incchk.result file if the check is OK. The ar link command depends on the result file.

Consider writing wrapper scripts of ar/link to merge the 2 processes.

Benefit:

  • Simpler dependency
  • Lesser output (hide the check command description echo)
  • Performance increasing by reduce ninja scheduling?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant