Skip to content
New issue

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

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

Already on GitHub? # to your account

lang/llvmir/parser: Factor out get_implicit_name(), use for implicit … #126

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pfalcon
Copy link
Contributor

@pfalcon pfalcon commented Feb 13, 2021

…labels.

LLVM IR has a habbit of using implicit names in various places, from
function params to basic block labels. In such cases, an autogenerated
"numbered name" is used.

Signed-off-by: Paul Sokolovsky pfalcon@users.sourceforge.net

…labels.

LLVM IR has a habbit of using implicit names in various places, from
function params to basic block labels. In such cases, an autogenerated
"numbered name" is used.

Signed-off-by: Paul Sokolovsky <pfalcon@users.sourceforge.net>
@pfalcon
Copy link
Contributor Author

pfalcon commented Feb 13, 2021

Continuation of #124

@codecov-io
Copy link

codecov-io commented Feb 13, 2021

Codecov Report

Merging #126 (2337e60) into master (aacfd18) will decrease coverage by 0.00%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #126      +/-   ##
==========================================
- Coverage   78.96%   78.96%   -0.01%     
==========================================
  Files         353      353              
  Lines       47323    47326       +3     
==========================================
+ Hits        37370    37372       +2     
- Misses       9953     9954       +1     
Impacted Files Coverage Δ
ppci/lang/llvmir/parser.py 12.64% <14.28%> (+0.10%) ⬆️
ppci/lang/tools/lr.py 96.49% <0.00%> (+0.38%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aacfd18...2337e60. Read the comment docs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants