erichkeane marked an inline comment as done. erichkeane added a comment. In D150875#4362434 <https://reviews.llvm.org/D150875#4362434>, @aaron.ballman wrote:
> Precommit CI found various failures that need to be addressed. Also, have you > tested to see what breaks in llvm-test-suite? Woops! I must have forgotten to re-make my diff before submitting. I fixed those in the patch earlier. I have not tested this against the llvm-test-suite as I'm not thrilled having to figure out how to build that again :) But I guess I should make a companion patch ahead of time. Last time however I don't recall us needing to add the flag for the warning however. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150875/new/ https://reviews.llvm.org/D150875 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits