[PATCH] D65575: [analyzer] Mention whether an event is about a condition in a bug report part 1

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 20 12:29:59 PDT 2019


NoQ added a comment.

In D65575#1621425 <https://reviews.llvm.org/D65575#1621425>, @Szelethus wrote:

> I think it isn't crucial of getting rid of the "The" prefix, if we append ", which participates in a condition later" (which sounds so much better than what I added in this patch), so maybe changing `WillBeUsedForACondition` to that would be good enough.


I guess let's just land that variant! I'm also not super worried about "The" here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65575





More information about the cfe-commits mailing list