[PATCH] D69950: Reapply "Fix crash on switch conditions of non-integer types in templates"

Elizabeth Andrews via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 8 14:33:51 PST 2019


eandrews added a comment.

Thank you for letting me know. The patch has been reverted.

The fail is a result of different diagnostics thrown on Windows and Linux. It looks like clang passes -fdelayed-template-parsing by default on Windows, which I assume is the reason why the diagnostic is not generated on Windows. I'm not entirely sure how different diagnostics in various targets are handled by clang-tidy. I'll push a new patch once I take a closer look.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69950





More information about the cfe-commits mailing list