Skip to content

Conversation

@thurstond
Copy link
Contributor

The heuristic handler for llvm.reverse is incorrect because it doesn't reverse the shadow.

Forked from llvm/test/CodeGen/X86/bitreverse.ll

The heuristic handler for llvm.reverse is incorrect because it doesn't reverse the shadow.

Forked from llvm/test/CodeGen/X86/bitreverse.ll
@thurstond thurstond merged commit ea9e174 into llvm:main Feb 4, 2025
9 checks passed
thurstond added a commit to thurstond/llvm-project that referenced this pull request Feb 4, 2025
llvm.bitreverse was incorrectly handled by the heuristic handler,
because it did not reverse the bits of the shadow.

This updates the instrumentation to use the handler from
llvm#114490 and  updates the test from llvm#125592
thurstond added a commit that referenced this pull request Feb 4, 2025
llvm.bitreverse was incorrectly handled by the heuristic handler,
because it did not reverse the bits of the shadow.

This updates the instrumentation to use the handler from
#114490 and updates the test
from #125592
github-actions bot pushed a commit to arm/arm-toolchain that referenced this pull request Feb 4, 2025
…dow (#125606)

llvm.bitreverse was incorrectly handled by the heuristic handler,
because it did not reverse the bits of the shadow.

This updates the instrumentation to use the handler from
llvm/llvm-project#114490 and updates the test
from llvm/llvm-project#125592
Icohedron pushed a commit to Icohedron/llvm-project that referenced this pull request Feb 11, 2025
The heuristic handler for llvm.reverse is incorrect because it doesn't
reverse the shadow.

Forked from llvm/test/CodeGen/X86/bitreverse.ll
Icohedron pushed a commit to Icohedron/llvm-project that referenced this pull request Feb 11, 2025
…25606)

llvm.bitreverse was incorrectly handled by the heuristic handler,
because it did not reverse the bits of the shadow.

This updates the instrumentation to use the handler from
llvm#114490 and updates the test
from llvm#125592
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