[cfe-commits] r133708 strikes me again? - the case clang stalls

Jay Foad jay.foad at gmail.com
Tue Aug 9 03:40:06 PDT 2011


On 9 August 2011 06:58, John McCall <rjmccall at apple.com> 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.




More information about the cfe-commits mailing list