StoeckOverflow wrote: Thanks, updated the patch to address the latest feedback.
The branch is now rebased on current `main`, so the volatile/nullability selector cleanup from the prequel PR is used directly here. I also switched the whitespace collapse to `SplitString`/`join`, made `APINotesParameterSelector` a type alias, added direct normalizer coverage, covered the member-function-pointer and nested pointer const cases, simplified the unit-test helper API, and kept `getParameterSelectorRefs` as a local adapter because the writer APIs still take `ArrayRef<StringRef>`. I also added the narrow `unsigned` -> `unsigned int` spelling normalization, but left `char` / `signed char` / `unsigned char` distinct since that would go beyond lexical selector normalization. https://github.com/llvm/llvm-project/pull/213043 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
