xazax.hun added a comment.

In https://reviews.llvm.org/D46234#1082307, @rsmith wrote:

> Can you say something about why you want to track this?


The original motivation was to fix this false positive in clang tidy: 
https://reviews.llvm.org/D46027

> Perhaps we should just use a null pointer for the not-instantiated arm of an 
> `if constexpr` instead, rather than tracking a flag on an empty statement 
> that is meaningless for real empty statements.

I will look into that, thanks. I wonder if any of the code assumes that the 
than branch always present.


Repository:
  rC Clang

https://reviews.llvm.org/D46234



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D46234: M... Gábor Horváth via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D462... Gábor Horváth via Phabricator via cfe-commits
    • [PATCH] D462... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to