simon_tatham added a comment.

Hmmm. I take your point, and so I backed out my local version of this patch in 
order to try to solve the problem a different way in overload resolution.

But in fact overloading on different vector types already seems to work fine: 
my in-progress patch series still passes all the clang tests even without this 
change to the defaults. So unless there's been a fix to overload resolution 
since I started developing all of this (I looked, and didn't find one), I can't 
quite work out why I needed this patch in the first place now. It may be that 
it's completely unnecessary, and that I managed to fool myself early in 
development by misinterpreting some other kind of mistake I'd made.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67160



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

Reply via email to