[compiler-rt] [profile] Fix bounds checks in profile merging (PR #118782)

Ellis Hoag via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 13:44:52 PST 2024


ellishg wrote:

> > For bound-free arithmetic, shall we cast to uintptr_t, ultimately?
> 
> I'm not sure I get what you mean here -- are you suggesting we should do everything in uintptr rather than const char and then only cast to a pointer type when doing accesses?

I agree with @chapuni, it seems cleaner to make the type of `SrcBitmapStart` (and similar variables) `uintptr_t` in the first place.

https://github.com/llvm/llvm-project/pull/118782


More information about the llvm-commits mailing list