Skip to content

Commit

Permalink
GitHub Actions: don't execute building ctags executable with -O0 option
Browse files Browse the repository at this point in the history
https://www.valgrind.org/docs/manual/quick-start.html

    Compile your program with -g to include debugging information so that
    Memcheck's error messages include exact line numbers.  Using -O0 is
    also a good idea, if you can tolerate the slowdown.  With -O1 line
    numbers in error messages can be inaccurate, although generally
    speaking running Memcheck on code compiled at -O1 works fairly well,
    and the speed improvement compared to running -O0 is quite
    significant.  Use of -O2 and above is not recommended as Memcheck
    occasionally reports uninitialised-value errors which don't really
    exist.

Suggested by @leleliu008 at universal-ctags#2747
  • Loading branch information
masatake committed Dec 19, 2020
1 parent 02e08b8 commit 1213b55
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/testing-with-vlagrind.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
branches: [ master ]

jobs:
testing-O0_fixme:
testing:

runs-on: ubuntu-latest

Expand Down

0 comments on commit 1213b55

Please sign in to comment.