Andres-Salamanca 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?

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