thesamesam added a comment.

Confirmed this patch fixes the miscompile.

Right. While the spec side needs discussion (and almost certainly addressing!), 
I don't think anyone in the real world is expecting the Clang behaviour vs the 
GCC one. The Clang one as-is is the one that's less conservative, as 
aggressively imposing Noreturn can only lead to something breaking.

I don't have ICC available to test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140868

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

Reply via email to