[PATCH] D50524: [Hexagon] Replace fatal error with remark in HexagonISelLowering

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

It might be worth discussing on llvmdev the question of whether it makes sense to add a flag to clang to control general undefined-behavior diagnostics, as a sort of best-effort thing when the optimizer finds it.  We have lib/Analysis/Lint.cpp, but there isn't any clang flag to turn it on.  Undefined behavior diagnostics are not something that would ever be reliable in the sense of consistently printing the same diagnostics across compiler versions/optimization levels, and it would always have false positives, so I don't think we would ever turn it on by default.  But it could be useful to help users figure out "why did my code disappear" when the compiler generates something unexpected.


Repository:
  rL LLVM

https://reviews.llvm.org/D50524