[PATCH] D61151: [SimpligyCFG] NFC, remove GCD that was only used for powers of two
    Shawn Landden via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Apr 30 07:00:27 PDT 2019
    
    
  
shawnl marked an inline comment as done.
shawnl added inline comments.
================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:5563
+    for (auto &V : Values)
+      V = (int64_t)((uint64_t)V >> Shift);
 
----------------
jmolloy wrote:
> I'd recommend just switching to using APInt. That's what it's for after all - making things like this self-explaining.
I remove your signed interpretation in a later patch.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61151/new/
https://reviews.llvm.org/D61151
    
    
More information about the llvm-commits
mailing list