WenleiHe wrote:

> Hi, Wenlei, I just updated the patch to sync with LLVM head. You can try with 
> option "--mllvm -enable-sampled-instr=true". I resumed work on this from the 
> last few weeks and I'm testing it with some internal benchmarks. One thing 
> that we noticed is that this patch can increase the text size quite a bit and 
> sometimes hits the 2GB relocation limit for large programs. I am thinking of 
> doing a function level sampling so that the text size increase would be 
> minimal. Please let me know if you have any problems with the patch and keep 
> us updated about the performance.
> […](#)
> -Rong
> On Fri, Jan 19, 2024 at 6:51 PM wenlei ***@***.***> wrote: Any follow ups on 
> this patch? Once it's in we'd like to give it a try as well. — Reply to this 
> email directly, view it on GitHub <[#69535 
> (comment)](https://github.com/llvm/llvm-project/pull/69535#issuecomment-1901669781)>,
>  or unsubscribe 
> <https://github.com/notifications/unsubscribe-auth/AOI42XXW4DZVXN42HYPHGUTYPMWKZAVCNFSM6AAAAAA6GII2ZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBRGY3DSNZYGE>
>  . You are receiving this because you authored the thread.Message ID: 
> ***@***.***>

Thanks. Will do! 

https://github.com/llvm/llvm-project/pull/69535
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to