[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