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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 19 09:08:45 PDT 2019


aaron.ballman added a comment.

In D63423#1547244 <https://reviews.llvm.org/D63423#1547244>, @regehr wrote:

> In D63423#1547216 <https://reviews.llvm.org/D63423#1547216>, @xbolva00 wrote:
>
> > The only remaining question is, as your said, whether or not to diagnose xors in macros. @regerh @jfb ?
>
>
> IMO it's better to not warn about xors in macros-- I like the current tradeoffs.


Do you mind explaining why? I would have expected to diagnose an xor within a macro because the code is just as wrong there, but maybe I'm missing something.


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

https://reviews.llvm.org/D63423





More information about the cfe-commits mailing list