rkirsling wrote:

I believe I'd need somebody to hit the merge button for me regardless, so I 
think we can just merge whichever one and close the other. 😅 The two approaches 
are mostly the same, I just figured that the surrounding code was consistent in 
using a literal `":"`, while there doesn't seem to be a precedent for using 
`tok::getPunctuatorSpelling` with a constant.

Either way, sorry to cause a run-around!

https://github.com/llvm/llvm-project/pull/144068
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to