hans added a comment.

In D59038#1422890 <https://reviews.llvm.org/D59038#1422890>, @rsmith wrote:

> LGTM, and I think this is safe enough to take for Clang 8.


Do you think the severity is high enough to spin another release candidate?

My concern is that since this didn't show up in testing until now, which I 
guess is quite a while after it regressed, maybe it's not important enough to 
delay the already late release? Also it sounds like it isn't completely fixed 
yet..


Repository:
  rC Clang

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

https://reviews.llvm.org/D59038



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

Reply via email to