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

TRIL parser generator fails on RHEL #4489

Closed
dibyendumajumdar opened this issue Oct 20, 2019 · 4 comments · Fixed by #4345
Closed

TRIL parser generator fails on RHEL #4489

dibyendumajumdar opened this issue Oct 20, 2019 · 4 comments · Fixed by #4345

Comments

@dibyendumajumdar
Copy link
Contributor

Hi,

I get this error:

[ 96%] [YACC][tril_parser] Building parser with YACC
cd /home/d/github/omr/fvtest/tril/tril && /usr/bin/yacc --defines=/home/d/github/omr/build/fvtest/tril/tril/tril.parser.h -o /home/d/github/omr/build/fvtest/tril/tril/tril.parser.c tril.y
Usage: /usr/bin/yacc [options] filename

Options:
  -b file_prefix        set filename prefix (default "y.")
  -d                    write definitions (y.tab.h)
  -i                    write interface (y.tab.i)
  -g                    write a graphical description
  -l                    suppress #line directives
  -o output_file        (default "y.tab.c")
  -p symbol_prefix      set symbol prefix (default "yy")
  -P                    create a reentrant parser, e.g., "%pure-parser"
  -r                    produce separate code and table files (y.code.c)
  -s                    suppress #define's for quoted names in %token lines
  -t                    add debugging support
  -v                    write description (y.output)
  -V                    show version information and exit
make[2]: *** [fvtest/tril/tril/CMakeFiles/tril.dir/build.make:62: fvtest/tril/tril/tril.parser.c] Error 1
make[2]: Leaving directory '/home/d/github/omr/build'
make[1]: *** [CMakeFiles/Makefile2:4923: fvtest/tril/tril/CMakeFiles/tril.dir/all] Error 2
make[1]: Leaving directory '/home/d/github/omr/build'
make: *** [Makefile:141: all] Error 2
@Leonardo2718
Copy link
Contributor

I have seen a similar error before. The yacc parser is being removed in favour of a hand-written parser in #4345, so that should take care of this problem. 🙂

@dibyendumajumdar
Copy link
Contributor Author

Yes I saw that ... Can it be merged asap please? Getting rid of yacc and lex dependency would be very welcome.

@0xdaryl
Copy link
Contributor

0xdaryl commented Oct 23, 2019

Should this issue be closed with a wontfix label then as it is superseded by #4345 ?

@Leonardo2718
Copy link
Contributor

#4345 has now been merged so this should no longer be an issue. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants