================
@@ -185,7 +185,7 @@ class ResultBuilder {
 
   /// Overloaded C++ member functions found by SemaLookup.
   /// Used to determine when one overload is dominated by another.
-  llvm::DenseMap<std::pair<DeclContext *, /*Name*/uintptr_t>, ShadowMapEntry>
+  llvm::DenseMap<std::pair<DeclContext *, /*Name*/ uintptr_t>, ShadowMapEntry>
----------------
timon-ul wrote:

I see a lot of whitespace changes in this file, note that this is not desired 
as you did not change any logic in these lines. You most likely got a complaint 
by the checkers about formatting, but these checkers only check your changed 
lines for failed formatting and thus you should only fix it in those lines, not 
the whole file. These formatting rules have grown over time and there is no 
desire to just reformat the whole codebase, since this will create more effort 
in tracking who actually touched code logic last. So please undo these 
formattings in the unrelated sections.

https://github.com/llvm/llvm-project/pull/197139
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to