Andy Kaylor comments

Results 6 comments of


                                            Andy Kaylor

inconsistency alignment of store instructions

Fixed by https://github.com/llvm/clangir/pull/1637

Add support for array new

I'm starting to work on this.

[CIR] Add default alignment to createStore

This has gotten hopelessly lost in rebasing. I came across this PR while looking for issues filed against this problem. I'm working on a similar solution.

Use `void (ptr)` on `LLVM::GlobalOp` when available in LLVM dialect to represent aliased functions

It looks like you implemented the necessary support in the LLVM dialect, but we still aren't lowering to aliases in CIR. I'm working on a patch that implements this for...

Use `void (ptr)` on `LLVM::GlobalOp` when available in LLVM dialect to represent aliased functions

I think https://github.com/llvm/clangir/pull/1739 addresses this issue.

ISel sinks math library calls below fesetenv

Yes, @topperc is correct. Per the C standard, access to the floating-point environment is undefined unless the fenv_access state is ON. The default state is implementation defined, but for clang...