-
Notifications
You must be signed in to change notification settings - Fork 26
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
llcppsymg:refine class fetch,test case #788
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #788 +/- ##
==========================================
+ Coverage 97.47% 97.52% +0.04%
==========================================
Files 20 20
Lines 5157 5769 +612
==========================================
+ Hits 5027 5626 +599
- Misses 110 123 +13
Partials 20 20 ☔ View full report in Codecov by Sentry. |
should resolve conflicts. |
9b75624
to
dc2e139
Compare
resolve conflicts , and need merge the #757 for llcppsigfetch & gogensig |
70c0de5
to
f253e79
Compare
7da1ba3
to
6f55f2f
Compare
4f90200
to
48acd52
Compare
ready for review. @xushiwei |
8e7e36b
to
7753f01
Compare
llcppsymg:headerpath test
llcppsymg:symbo test llcppsymg:exist symb file test llcppsymg:GenSymbolTabledata llcppsymg:GenSymbolTableData test llcppsymg:full symg operation test
llcppsymg:clangutil test
7753f01
to
da615cc
Compare
[Git-flow] Hi @luoliwoshang, There are some suggestions for your information: Rebase suggestions
Which seems insignificant, recommend to use For other If you have any questions about this comment, feel free to raise an issue here: |
[wip] fix #830
linux
/etc/ld.so.conf.d
's subdirIt can pass tests normally in both Linux and macOS environments.
However, in actual operation on Linux, the part calling nm throws the following error. Investigation reveals that it's caused by exec.Command. Running
cmptest/osexec
on Linux also produces the corresponding issue: