[PATCH] D20339: Update clang for D20260

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 14 14:01:27 PDT 2016


rsmith accepted this revision.
rsmith added a reviewer: rsmith.
rsmith added a comment.
This revision is now accepted and ready to land.

Adding `-cl-opt-disable` to the tests seems fine as an alternative short-term approach; LGTM with that approach (either as part of this change or a separate change before/after). In the long term, I agree that it would be better if the driver specified the relevant -O flag here.


http://reviews.llvm.org/D20339





More information about the cfe-commits mailing list