https://github.com/sdesmalen-arm approved this pull request.
Assuming the tests pass after changing `BuiltinType` to a `std::optional`, the patch looks fine to me. https://github.com/llvm/llvm-project/pull/75487 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits