second-state / SOLL

SOLL is a new compiler for generate Ewasm from solidity and yul. See a demo here: https://asciinema.org/a/ezJqNLicn5fya02zwu4VXIo8a
https://www.secondstate.io/
378 stars 24 forks source link

[AST] Enhance Struct Decl #117

Open LFsWang opened 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #117 (d748632) into master (821b1f6) will decrease coverage by 1.74%. The diff coverage is 61.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   66.47%   64.73%   -1.75%     
==========================================
  Files          98      106       +8     
  Lines       13679    14095     +416     
  Branches     2041     2162     +121     
==========================================
+ Hits         9093     9124      +31     
- Misses       3567     3888     +321     
- Partials     1019     1083      +64     
Impacted Files Coverage Δ
include/soll/AST/Stmt.h 98.03% <0.00%> (-1.97%) :arrow_down:
include/soll/Sema/Sema.h 90.00% <ø> (ø)
include/soll/Sema/GlobalContent.h 21.42% <21.42%> (ø)
lib/AST/DeclVisitor.cpp 80.85% <33.33%> (-3.60%) :arrow_down:
lib/AST/Expr.cpp 60.21% <33.33%> (-5.38%) :arrow_down:
include/soll/AST/Decl.h 80.58% <47.82%> (-9.55%) :arrow_down:
include/soll/Sema/DeclarationContainer.h 50.00% <50.00%> (ø)
lib/AST/Decl.cpp 67.96% <51.72%> (-3.94%) :arrow_down:
include/soll/Sema/NameAndTypeResolver.h 55.55% <55.55%> (ø)
include/soll/Sema/ReferenceResolver.h 57.14% <57.14%> (ø)
... and 24 more

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 821b1f6...d748632. Read the comment docs.

hydai commented 3 years ago

Please rebase to the latest master

lgtm-com[bot] commented 3 years ago

This pull request introduces 3 alerts when merging c3c0042ee907dbfea8d86e6c40a75b6d94fefb22 into 821b1f64ff622e4e29bb9165ca4eaf4407bdc687 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 63132918aa69a5349e1d003c2dbd6d3ae81e6b4a into 821b1f64ff622e4e29bb9165ca4eaf4407bdc687 - view on LGTM.com

new alerts: