The first patch is OK. The comment in EmitBranchOnBoolExpr says: "Based on the condition, this might try to simplify the codegen of the conditional based on the branch.". Can we add a testcase where such an optimization is present?
On 29 October 2013 17:52, Justin Bogner <[email protected]> wrote: > Two patches that clarify the logic to emit for/for-range statements a > bit. There shouldn't be any meaningful functionality change. > > Okay to commit these? > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
