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



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

Reply via email to