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

The simple loop fails #552

Closed
EugeniyPetrov opened this issue Aug 15, 2012 · 3 comments
Closed

The simple loop fails #552

EugeniyPetrov opened this issue Aug 15, 2012 · 3 comments

Comments

@EugeniyPetrov
Copy link

Hello again,

I'm playing with hip-hop on ubuntu 12.04. Trying to compile the following code:

$ cat hello.php

$ ~/dev/hiphop-php/src/hphp/hphp --keep-tempdir=1 --log=3 hello.php --target=cpp --format=exe
running hphp...
creating temporary directory /tmp/hphp_wYXHi3 ...
parsing inputs...
parsing inputs took 0'00" (3 ms) wall time
pre-optimizing...
pre-optimizing took 0'00" (1 ms) wall time
analyze includes...
analyze includes took 0'00" (0 ms) wall time
inferring types...
inferring types took 0'00" (1 ms) wall time
post-optimizing...
post-optimizing took 0'00" (1 ms) wall time
creating CPP files...
creating CPP files took 0'00" (40 ms) wall time
compiling and linking CPP files...
/tmp/hphp_wYXHi3/sys/main.cpp: In function \xe2\x80\x98void reExec(const char_, char__)\xe2\x80\x99:\n/tmp/hphp_wYXHi3/sys/main.cpp:95:25: warning: ignoring return value of \xe2\x80\x98char_ realpath(const char_, char_)\xe2\x80\x99, declared with attribute warn_unused_result [-Wunused-result]\n
compiling and linking CPP files took 0'57" (57429 ms) wall time
all files saved in /tmp/hphp_wYXHi3 ...
running hphp took 0'57" (57605 ms) wall time

Warning appears but output file executes:

$ /tmp/hphp_wYXHi3/program --file=hello.php
done

If I change constant to 1024 code still compiles with the same warning but fails on execute.

$ /tmp/hphp_VvdG0N/program --file=hello.php
program: /home/eug/dev/hiphop-php/src/runtime/base/types.h:396: void HPHP::check_request_timeout(HPHP::ThreadInfo*): Assertion `false' failed.
Core dumped: Aborted
Aborted (core dumped)

Similar problem I had early on another machine and another instance of hip-hop. But it worked with 65535 and failed with 65536.

I do appreciate for any help.
Thank you!

@thedjnK
Copy link
Contributor

thedjnK commented Aug 17, 2012

Tested and get the same error as soon as it hits 1024, not reproducable in HHVM.

@d3xt3r01
Copy link

d3xt3r01 commented Sep 2, 2012

Any updates on this issue ? :/

@ptarjan
Copy link
Contributor

ptarjan commented May 13, 2013

We're closing out all bugs older than 2 months. http://www.hiphop-php.com/wp/?p=575

If this is still an issue, please re-open it, and in order of goodness:

  1. Give detailed repro steps
  2. Write a test case in hphp/tests/quick (run it with hphp/tests/run) and send the pull request
  3. Fix it in a pull request

@ptarjan ptarjan closed this as completed May 13, 2013
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

No branches or pull requests

4 participants