https://github.com/wangleiat commented:

No issues overall, just one minor question:

According to the 
[LangRef,](https://llvm.org/docs/LangRef.html#function-attributes) the default 
value of "stack-probe-size" is 4096:

```
It takes one required integer value, which is 4096 by default.
```

However, on LoongArch, the kernel page size is typically 16K. If we still use 
4K here, could that introduce unnecessary store instructions during stack 
probing?

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

Reply via email to