Skip to content

Test bot force push - #12

Closed
PLukas2018 wants to merge 5 commits into
masterfrom
test-bot
Closed

PLukas2018 wants to merge 5 commits into
masterfrom
test-bot

Conversation

@PLukas2018

Copy link
Copy Markdown
Owner

No description provided.

Functions that are called inside the definitions of prefix and postfix
functions do not throw any exceptions. Therefore, we can make them
`noexcept` which could have positive impact on the performance and is
regarded as a good practice.
@PLukas2018

Copy link
Copy Markdown
Owner Author

\eqbench-test --help

@PLukas2018

Copy link
Copy Markdown
Owner Author

\evaluate-test --help

@diffkemp-dev-bot

Copy link
Copy Markdown
Usage: \evaluate [options]

Evaluator of pull requests

Options:
  --run <experiments...>     selection of experiments to be run (choices:
                             "eqbench", "rhel-sysctl", "rhel-functions")
  --pr-cmp-opt <options...>  option to add options for PR's `compare` command
  --cmp-opt <options...>     option to add options for `compare` command
  --rebuild                  rebuild snapshots for comparisons on PR
  -h, --help                 display help for command

@diffkemp-dev-bot-fork

Copy link
Copy Markdown
Usage: \evaluate [options]

Evaluator of pull requests

Options:
  --run <experiments...>     selection of experiments to be run (choices: "eqbench", "rhel-sysctl", "rhel-functions")
  --pr-cmp-opt <options...>  option to add options for PR's `compare` command
  --cmp-opt <options...>     option to add options for `compare` command
  --rebuild                  rebuild snapshots for comparisons on PR
  -h, --help                 display help for command

@PLukas2018

Copy link
Copy Markdown
Owner Author

\evaluate-test

@diffkemp-dev-bot

Copy link
Copy Markdown
error: unknown option '-test'

Usage: \evaluate [options]

Evaluator of pull requests

Options:
  --run <experiments...>     selection of experiments to be run (choices:
                             "eqbench", "rhel-sysctl", "rhel-functions")
  --pr-cmp-opt <options...>  option to add options for PR's `compare` command
  --cmp-opt <options...>     option to add options for `compare` command
  --rebuild                  rebuild snapshots for comparisons on PR
  -h, --help                 display help for command

@diffkemp-dev-bot-fork

Copy link
Copy Markdown

Error occurred while running evaluation.

@PLukas2018

Copy link
Copy Markdown
Owner Author

\evaluate-test

@diffkemp-dev-bot

Copy link
Copy Markdown
error: unknown option '-test'

Usage: \evaluate [options]

Evaluator of pull requests

Options:
  --run <experiments...>     selection of experiments to be run (choices:
                             "eqbench", "rhel-sysctl", "rhel-functions")
  --pr-cmp-opt <options...>  option to add options for PR's `compare` command
  --cmp-opt <options...>     option to add options for `compare` command
  --rebuild                  rebuild snapshots for comparisons on PR
  -h, --help                 display help for command

This commit adds a support for LLVM 19, the main differences in
comparison to the previous version are:

1. The removal of `equals` method from `StringRef` class.
2. The removal of `getPredicate` from `ConstExpr` class.
3. Various changes in includes, that require us to use more fine-grained
   include strategy.
4. The separation of some analysis functions and classes into a separate
   header.
5. The deprecation of debug intrinsics has begun.

The `equals` method was removed in this commit [1] and the equality
operator is in the code base from LLVM 3, so it should be safe to use.

The name of some passes has changed in [2]. Therefore, we have to check
for some of these passes and change their name in accordance to the used
LLVM version.

Because of that we have to check the version with which was the DiffKemp
SimpLL library built. This required addition of new function to the FFI.

The `getPredicate` method was removed from `ConstExpr` class in commit [3],
we have extracted the method for getting an instruction from expression
into a stand-alone function, but this method has changed upstream.
Therefore, I have extracted the new changed method into a new function.

The analysis classes and functions were moved into a new header in [4].

The LLVM aims to remove debug intrinsics, as explained in [5]. However,
we still rely on them in our tests. Therefore, we have to set the debug
type back to the old one as our code was affected by [6].

[1]: llvm/llvm-project@3fa409f
[2]: llvm/llvm-project@e390c22
[3]: llvm/llvm-project@deab451
[4]: llvm/llvm-project@a4cd981
[5]: https://llvm.org/docs/RemoveDIsDebugInfo.html
[6]: llvm/llvm-project@9997e03
Add additional entries to .gitignore for build files and test directories.
@PLukas2018 PLukas2018 closed this Oct 22, 2025
@PLukas2018
PLukas2018 deleted the test-bot branch October 22, 2025 07:40
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.

2 participants