NoQ added a comment.

> The analyzer test change maybe indicates we could simplify the analyzer code 
> a little with this fix? Apparently a hack was added to support lvalues in 
> initializers in r315750, but I'm not really familiar with the relevant code.

Nope, unfortunately, not much can be simplified. This hack is much deeper and 
covers many more cases than that and was added much earlier :) But thanks for 
attracting my attention to this!


Repository:
  rC Clang

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

https://reviews.llvm.org/D58069



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D58069: [... Eli Friedman via Phabricator via cfe-commits
    • [PATCH] D580... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D580... Eli Friedman via Phabricator via cfe-commits
    • [PATCH] D580... Eli Friedman via Phabricator via cfe-commits
    • [PATCH] D580... Artem Dergachev via Phabricator via cfe-commits

Reply via email to