kazutakahirata wrote: > LGTM, the `inline` seems unnecessary since `static` and `constexpr` should already imply `inline` on data members. Good point! Fixed. Thanks for the review as always! https://github.com/llvm/llvm-project/pull/161480