Skip to content

Conversation

ghehg
Copy link
Contributor

@ghehg ghehg commented Dec 2, 2024

Basically, for int type, the order of Ops is not the same as OG in the emitted LLVM IR. OG has constant as the second op position. See OG's order .

@ghehg ghehg changed the title [CIR][CodeGen][LowerToLLVM] Fix llvm lowering of cir.unary not [CIR][CodeGen][LowerToLLVM] Fix llvm lowering of cir.unary kind NOT Dec 2, 2024
@ghehg ghehg changed the title [CIR][CodeGen][LowerToLLVM] Fix llvm lowering of cir.unary kind NOT [CIR][CodeGen][LowerToLLVM] Fix llvm lowering of CIR UnaryOpKind_Not Dec 2, 2024
@ghehg ghehg marked this pull request as ready for review December 3, 2024 03:33
@ghehg ghehg added the IR difference A difference in ClangIR-generated LLVM IR that could complicate reusing original CodeGen tests label Dec 3, 2024
@bcardosolopes bcardosolopes merged commit 9a951af into llvm:main Dec 3, 2024
10 checks passed
lanza pushed a commit that referenced this pull request Mar 18, 2025
#1194)

Basically, for int type, the order of Ops is not the same as OG in the
emitted LLVM IR. OG has constant as the second op position. See [OG's
order ](https://godbolt.org/z/584jrWeYn).
terapines-osc-cir pushed a commit to Terapines/clangir that referenced this pull request Sep 2, 2025
llvm#1194)

Basically, for int type, the order of Ops is not the same as OG in the
emitted LLVM IR. OG has constant as the second op position. See [OG's
order ](https://godbolt.org/z/584jrWeYn).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IR difference A difference in ClangIR-generated LLVM IR that could complicate reusing original CodeGen tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants