aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Transforms/Utils/InlineFunction.cpp
diff options
context:
space:
mode:
authorRainer Orth <ro@gcc.gnu.org>2025-05-10 09:50:08 +0200
committerGitHub <noreply@github.com>2025-05-10 09:50:08 +0200
commit1b4f161978f334197eb6902872699d3e92060d66 (patch)
tree7ff0a7805e1862534d48818a858fa43dc80b79fe /llvm/lib/Transforms/Utils/InlineFunction.cpp
parentb7c449ac0b0c4ccbe99937052c9428960cea7664 (diff)
downloadllvm-1b4f161978f334197eb6902872699d3e92060d66.zip
llvm-1b4f161978f334197eb6902872699d3e92060d66.tar.gz
llvm-1b4f161978f334197eb6902872699d3e92060d66.tar.bz2
[OpenMP][test] Make %p 0x prefix optional (#138514)
Testing `openmp` on Solaris/amd64 shows a large number of failures, all due to the same issue: ``` # .---command stderr------------ # | openmp/runtime/test/ompt/misc/interoperability.cpp:67:16: error: CHECK-SAME: expected string not found in input # | // CHECK-SAME: parent_task_frame.reenter={{0x[0-f]+}} # | ^ # | <stdin>:5:101: note: scanning from here # | 281474976710658: ompt_event_parallel_begin: parent_task_id=281474976710659, parent_task_frame.exit=0, parent_task_frame.reenter=7fffbedffe90, parallel_id=281474976710661, requested_team_size=2, codeptr_ra=408b8e, invoker=2 ``` The testsuite expects pointers to be printed with a `0x` prefix when using the `%p` format, while Solaris `libc` just prints them in hex without a prefix. However, this difference is completely benign. ISO C (up to C23, 7.23.6.1) states ``` p The argument shall be a pointer to void or a pointer to a character type. The value of the pointer is converted to a sequence of printing characters, in an implementation-defined manner. ``` I saw two ways around this: - replace every instance of `%p` with a macro (`KMP_PTR_FMT`, defined as `"%p"` or `"0x%p" as appropriate), or - adjust the testsuite to make the `0x` prefix optional The second route seemed less intrusive and more readable, so that's what this patch does. While large, it's also completely mechanical. Tested on `amd64-pc-solaris2.11` and `x86_64-pc-linux-gnu`.
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
0 files changed, 0 insertions, 0 deletions