dblaikie added a comment.

Could the test be expanded to test more specifically than "does not crash"? 
Like is there some part of the output that can't be observed in any other test 
(that didn't/doesn't crash without this patch) - like the location emitted in 
some debug info, etc?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136807

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

Reply via email to