xlauko wrote:

> yeah that looks better! quick question why did you drop the type of 
> `dynAllocSize`? i noticed `llvm.alloca` uses `x` to separate it, in our case 
> i think keeping it with a `,` looks good: `%0 = cir.alloca "n" align(4) init 
> %sz : !u64i, !s32i -> !cir.ptr<!s32i>` what do you think?

I just forgot about it :D 

https://github.com/llvm/llvm-project/pull/198962
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to