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

Optim #104

Merged
merged 60 commits into from Feb 23, 2019
Merged

Optim #104

merged 60 commits into from Feb 23, 2019

Conversation

fennecdjay
Copy link
Member

Using computed gotos, fixing dynamicity.

@todo
Copy link

todo bot commented Feb 8, 2019

improve me

https://github.com/fennecdjay/Gwion/blob/1a1d163508bfaeac7b33b0f83c81b7e87a328c94/src/emit/emit.c#L1449-L1454


This comment was generated by todo based on a TODO comment in 1a1d163 in #104. cc @fennecdjay.

@fennecdjay
Copy link
Member Author

This pull request introduces 1 alert when merging d37335e into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 1 alert when merging 54b98a4 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging 2096e78 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging a06ee3e into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging 3b3eae4 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging dd68a11 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging 1630cfd into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging 24c6db5 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging f3960e4 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging a89005c into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging 1d6d8f4 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging ce5f9fb into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

This pull request introduces 2 alerts when merging c3b73f5 into b8ca5c8 - view on LGTM.com

new alerts:

  • 1 for Declaration hides parameter
  • 1 for Use of goto

Comment posted by LGTM.com

@fennecdjay
Copy link
Member Author

🎉
A big pull Request.

  • new VM, 3x faster
  • fix dinamicity
    and other fixes...
    😄

@fennecdjay fennecdjay merged commit d8884a1 into master Feb 23, 2019
@fennecdjay fennecdjay deleted the optim branch February 25, 2019 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant