manas added a comment.

I have made few changes:

1. The failed assertions <https://reviews.llvm.org/F19816216> due to comparison 
between different types have been fixed by converting all the Ranges to a given 
type. This will allow comparisons without throwing errors.

2. There was also a build error due to `explicit specialization in 
non-namespace scope`. This was fixed by @martong previously, but that patch led 
to the above mentioned bug. So I used @martong 's patch here to make GCC happy.

3. I have added a small check for comparison between different types.

https://reviews.llvm.org/D106102 differential contains the background story.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112621

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

Reply via email to