[cfe-dev] Patch: Implement break and continue

Anders Carlsson andersca at mac.com
Mon Jul 16 07:40:55 PDT 2007

15 jul 2007 kl. 22.02 skrev Chris Lattner:

> On Jul 15, 2007, at 9:05 PM, Anders Carlsson wrote:
>> Hello,
>> here's a patch that implements break and continue in for, while  
>> and do loops.
>> Since this is my first encounter ever with LLVM and cfe, the patch  
>> could very well be completely wrong :) I'd love any feedback it  
>> though.
> I have to say that this is *not* how I envisioned implementing  
> this: it's actually a much better approach.
> Overall, I really like the patch.  Some suggestions:
> 1. Instead of having separate break/continue stacks, I'd suggest  
> merging them and using a SmallVector<std::pair> (or a custom struct  
> instead of a pair if you prefer).  For switch stmts (when we  
> support them), we'd just use null as the continue point.

Thanks for the comments! Here's a new patch that addresses all of  
them and adds an extra assert that the break/continue stack is empty  
after generating the function body.

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: break-continue-in-loops-2.txt
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20070716/b8c1a37c/attachment.txt>
-------------- next part --------------

More information about the cfe-dev mailing list