ilya-biryukov added a comment.

> I had completely missed that there could be conflicting tokens when only 
> highlighting macro arguments as well. Added code to just remove conflicting 
> tokens.

Picking one of the highlightings looks fine, but we probably want to make sure 
it's deterministic. Given that we use `sort` now, I bet it won't be. Maybe 
include kinds into comparison as well? That's not a perfect solution, but would 
at least make sure the user-visible behavior is not random.
Could you add tests for that case too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64741/new/

https://reviews.llvm.org/D64741



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to