[clang] [Bounds Safety][NFC] Add `SemaBoundsSafety` class and move existing Sema checks there (PR #98954)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 17 06:56:56 PDT 2024


AaronBallman wrote:

> > Out of curiosity, how does the current division of Sema fit into the "unique language dialect" classification? I've noticed there are a bunch of architecture specific Sema classes (e.g. SemaRISCV, SemaX86) and those don't really fit the classification.
> 
> They are a different group of `Sema` parts, dedicated for backend-specific code. `SemaX86` and `SemaARM` serve as a good example of what a populated backend-specific part looks like.

Yup, and they're an example of where it helps to have layering (if parts of `SemaARM` need to call into `SemaX86`, it's good for that dependency to be more explicit).

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


More information about the cfe-commits mailing list