[PATCH] D36768: [test-suite] Add -i option to fpcmp to ignore whitespace changes.
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 21 09:31:34 PDT 2017
hfinkel added inline comments.
================
Comment at: tools/fpcmp.c:283
+ if (ignore_whitespace) {
+ if (skip_whitespace(&F1P, File1End) | skip_whitespace(&F2P, File2End))
+ continue;
----------------
Any particular reason you're using a bitwise instead of logical or. If not, I'd prefer `||` here.
https://reviews.llvm.org/D36768
More information about the llvm-commits
mailing list