mydeveloperday wrote: > > I don't understand why we need to refactor this? > > During the review for #77918 there was an ask to extract the refactor in a > separate PR. > > In #77918, I'm adding a couple new test fixtures and plan to use these > utility functions for verifying assumptions.
Am I missing someone but how do you plan on using these functions if they are in an anonymous namespace in the file and not part of a class where they are static? maybe I missed something. https://github.com/llvm/llvm-project/pull/78108 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
