[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