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

Reply via email to