[PATCH] D57914: [Driver] Allow enum SanitizerOrdinal to represent more than 64 different sanitizer checks, NFC.

pierre gousseau via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 21 09:14:26 PST 2019


pgousseau added a comment.

In D57914#1405955 <https://reviews.llvm.org/D57914#1405955>, @riccibruno wrote:

> More explicitly something like:
>
> in `Sanitizer.h`:
>
>   template <typename T = void> struct SanitizerMasks {
>     static const SanitizerMask SomeMask;
>     /* and so on for each mask*/
>   };
>  
>   template <typename T> const SanitizerMask SanitizerMasks<T>::SomeMask = the definition;
>
>
> And then you can write `SanitizerMasks<>::SomeMask` when you want to use this mask.


Sounds good to me, I will try that thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57914/new/

https://reviews.llvm.org/D57914





More information about the cfe-commits mailing list