[PATCH] D67160: [clang, ARM] Default to -fno-lax-vector-conversions in ARM v8.1-M.

[Simon Tatham via Phabricator via cfe-commits]

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