[PATCH] D63423: [Diagnostics] Diagnose misused xor as pow

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 19 10:30:05 PDT 2019


jfb added a comment.

What I meant with macros was that I don't think we should warn on:

  #define LEGIT(a, b) ({ work work work; a ^ b; work work work; })
  
  LEGIT(10, 5);

If the constants are inline in the macros then sure, warn there. Basically: if you literally wrote `CONSTANT ^ CONSTANT` and it looks fishy, let's warn. If token-pasting wrote it for you and it looks fishy, let's not warn.


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

https://reviews.llvm.org/D63423





More information about the cfe-commits mailing list