-
Notifications
You must be signed in to change notification settings - Fork 397
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
Use custom function to format TRIL code #5587
Open
janvrany
wants to merge
22
commits into
eclipse-omr:master
Choose a base branch
from
janvrany:pr/use-custom-tril-format
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Dec 15, 2023
-
Introduce new custom function to format TRIL code
This commit introduces new template function `Tril::format()` that is to be used to format TRIL code in tests designed as drop-in replacement for `std::snprintf()` except that: 1. support only a (used) subset of *printf-like conversion specifiers, 2. "modifiers" like l, ll, h, hh, j, I, I32 and I64 are ignored and 3. used conversion specifier and a static type of actual parameter must match (it is not possible to format `double` using `%X` conversion) Use of a template function and static types rather than *printf-like conversion flags allows `Tril::format()` to be used in another templates and thus will allow to further simplify test creation. Moreover, this makes TRIL formatting independent on `std::snprintf()` which is known to behave differently on different systems. Therefore, this commit introduces indirection required to fix eclipse-omr#5183 and eclipse-omr#5324.
Configuration menu - View commit details
-
Copy full SHA for b7b437f - Browse repository at this point
Copy the full SHA b7b437fView commit details -
Add poor man's replacement for std::make_signed
Since XLC compiler used at a time on AIX and ZOS does not support std::make_signed, this commit provides simplified drop-in replacement for use in reinterpret_as_signed(). Once XLC compiler on AIX and ZOS is upgraded to support std::make_signed, this simplified version can be removed in favour of standard version.
Configuration menu - View commit details
-
Copy full SHA for 0ec553b - Browse repository at this point
Copy the full SHA 0ec553bView commit details -
Configuration menu - View commit details
-
Copy full SHA for f14bdc2 - Browse repository at this point
Copy the full SHA f14bdc2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 671ba7a - Browse repository at this point
Copy the full SHA 671ba7aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9d2ff16 - Browse repository at this point
Copy the full SHA 9d2ff16View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8671282 - Browse repository at this point
Copy the full SHA 8671282View commit details -
Configuration menu - View commit details
-
Copy full SHA for e8cfd53 - Browse repository at this point
Copy the full SHA e8cfd53View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4db9382 - Browse repository at this point
Copy the full SHA 4db9382View commit details -
Configuration menu - View commit details
-
Copy full SHA for 995ea89 - Browse repository at this point
Copy the full SHA 995ea89View commit details -
Configuration menu - View commit details
-
Copy full SHA for 22072f1 - Browse repository at this point
Copy the full SHA 22072f1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3308657 - Browse repository at this point
Copy the full SHA 3308657View commit details -
Configuration menu - View commit details
-
Copy full SHA for 204b896 - Browse repository at this point
Copy the full SHA 204b896View commit details -
Configuration menu - View commit details
-
Copy full SHA for e9251af - Browse repository at this point
Copy the full SHA e9251afView commit details -
Configuration menu - View commit details
-
Copy full SHA for f7503d7 - Browse repository at this point
Copy the full SHA f7503d7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9b01afe - Browse repository at this point
Copy the full SHA 9b01afeView commit details -
Configuration menu - View commit details
-
Copy full SHA for de4668f - Browse repository at this point
Copy the full SHA de4668fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 18e04eb - Browse repository at this point
Copy the full SHA 18e04ebView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0760347 - Browse repository at this point
Copy the full SHA 0760347View commit details -
Configuration menu - View commit details
-
Copy full SHA for 505dbdd - Browse repository at this point
Copy the full SHA 505dbddView commit details -
Convert SimpifierFoldAbsNegTest tests to use Tril::format() instead o…
…f std::snprintf()
Configuration menu - View commit details
-
Copy full SHA for 13024c5 - Browse repository at this point
Copy the full SHA 13024c5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 47c882c - Browse repository at this point
Copy the full SHA 47c882cView commit details -
Configuration menu - View commit details
-
Copy full SHA for ad7275b - Browse repository at this point
Copy the full SHA ad7275bView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.