[PATCH] D17375: Add parentheses around arithmetic in operand of '|' in llvm-dwp.cpp.
    Benjamin Kramer via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu Feb 18 05:14:23 PST 2016
    
    
  
bkramer added a comment.
I think this is the wrong solution. Per http://www.dwarfstd.org/ShowIssue.php?issue=140421.1
  2.  Calculate a secondary hash H' = (((S >> 32) & MASK(k)) | 1).
  [...]
  4.  Let H = (H + H') modulo M. Repeat at Step 3.
So the OR has to happen before the ADD.
http://reviews.llvm.org/D17375
    
    
More information about the cfe-commits
mailing list