[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