On 9 August 2011 06:58, John McCall <[email protected]> wrote:
> I think your patch to LLVM is a good idea;  replaceAllUsesWith should
> not be assuming a fully-formed AST.  As a slight optimization, I would
> suggest doing this instead:
>  if (Succ->empty()) continue;
> Jay, does that seem reasonable?

Sure, I'm fine with either your or Takumi's fix. Thanks for taking the
time to investigate.

Jay.

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to