Bruno Cardoso Lopes
Bruno Cardoso Lopes
Since this is basically all types, we could probably get this from tablegen. @sitio-couto recently removed a tablegen constraint for checking that would have been helpful there. Let me get...
> My main issue with this conditional is that it omits the `cir.scope` op in certain scenarios. A `cir.if`, for example, will not have an explicitly defined `cir.scope` operation in...
Needed for passing info down to LLVM lowering but should build a base for writing AA pass in CIR
Unfortunately, mutable types do not support aliases yet, but there is a fix for this in review: https://github.com/llvm/llvm-project/pull/66663. At some point, we will likely be able to use aliases on...
Got it. The [doc](https://llvm.github.io/clangir/Dialect/ops.html#cirscope-cirscopeop) seems to be a little outdated. _Originally posted by @Kuree in https://github.com/llvm/clangir/pull/262#discussion_r1329448293_
Driver isn't doing the proper job of giving precedence to the last specified one (and the message is also pretty broken). ``` clang++ -std=c++20 -target arm64-apple-macosx14.0.0 -Wno-unused-value -fexceptions -c lambda.cpp...
#731 adds CIRGen support for non-constant memory orders all the way down to LLVM. To keep CIR a bit more simple and easier to analyze, we could change atomic operations...
One example I just tried and crashes: `__atomic_thread_fence(__ATOMIC_SEQ_CST);`
Inspired by @Lancern work on providing some ASM guideline goodness, I created some guidelines for `auto` and `camelCase` usage, see it here: https://llvm.github.io/clangir/GettingStarted/coding-guideline.html However, we still need to encode the...
If we look at how LLVM currently codegen for this, we get a `add nsw` instead of a plain `add`, see https://godbolt.org/z/xdevh38r8. There are probably other differences too. _Originally posted...