NoQ added a comment.

I'd rather not abandon this patch, because it looks like a strict improvement 
over the lack of condition tracking, and it might as well still be an 
improvement over "zealous" condition tracking, as my counterexample is fairly 
artificial. It indicates that a slightly more sophisticated algorithm is 
necessary (i'm not sure if it's single-pass or even linear). But i'll be 
perfectly happy with simply adding it as a FIXME test.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64271/new/

https://reviews.llvm.org/D64271



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to