-
Notifications
You must be signed in to change notification settings - Fork 5
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
llcppg:support split run symbol gen & code gen #142
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #142 +/- ##
=======================================
Coverage 98.12% 98.12%
=======================================
Files 17 17
Lines 2132 2132
=======================================
Hits 2092 2092
Misses 28 28
Partials 12 12 ☔ View full report in Codecov by Sentry. |
e9a08d1
to
718434e
Compare
5cd34b7
to
f95901e
Compare
f95901e
to
17befc6
Compare
17befc6
to
0f1e8f6
Compare
if !codeGen { | ||
err = llcppsymg(b) | ||
check(err) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
建议添加一个RunMode枚举
llcppg.go
Outdated
go llcppsigfetch(b, w) | ||
if !symbGen { | ||
r, w := io.Pipe() | ||
go llcppsigfetch(b, w) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
添加一个RunMode枚举
llcppg -symbgen
to only trigger llcppsymg to generate symbol table(llcppg.symb.json)llcppg -codegen
to only generate go binding (llcppsigfetch & gogensig)llcppg [-vsymg|-vgogen|-vfetch]
to control log for llcppsymg,llcppsigfetch,gogensig