Alan Li

Results 35 comments of Alan Li

They both generate same structure of LLVM IR, except: In the demote path, generated IR contains a load of `i32`: ``` %60 = llvm.getelementptr %31[%59] : (!llvm.ptr, i64) -> !llvm.ptr,...

Attaching dumped IR files. This one enables `--iree-input-demote-i64-to-i32` which causes value not matching: [demote_to_i32.mlir.txt](https://github.com/user-attachments/files/16703658/demote_to_i32.mlir.txt) This one does not have the option and works fine: [no_demote_to_i32.mlir.txt](https://github.com/user-attachments/files/16703661/no_demote_to_i32.mlir.txt)

@MaheshRavishankar See the dump file: [out_mlir.txt](https://github.com/user-attachments/files/16703621/out_mlir.txt)

Confirmed the issue no longer persists on main branch 56ecef7f .

Problematic `memref.alloca`: ``` %54 = "memref.alloca"(%45) : (index) -> memref

@MaheshRavishankar see attached: [18297.mlir.txt](https://github.com/user-attachments/files/16703692/18297.mlir.txt)

Confirm the issue is gone along with #18296 in latest main branch.

Moving this out of CPU project board. @pashu123 still need to fuse pack+unpack, per @MaheshRavishankar

@zjgarvey @rsuderman Can you provide some context input on this issue? As @pdhirajkumarprasad mentioned, it is possible that it is caused by it.

@zjgarvey So @pashu123 has approved the PR, and it looks clear. please proceed.