[PATCH] Extend EarlyCSE to handle basic cases from JumpThreading	and CVP
    Philip Reames 
    listmail at philipreames.com
       
    Wed May 13 16:13:41 PDT 2015
    
    
  
================
Comment at: lib/Transforms/Scalar/EarlyCSE.cpp:490
@@ +489,3 @@
+                  ToUpdate.push_back(IU);
+            for (Instruction *UserI : ToUpdate) {
+              DEBUG(dbgs() << "EarlyCSE CVP: Replace dominated use of '"
----------------
dberlin wrote:
> Can you common this part with GVN's replaceAllDominatedUsesWith?
> 
> (IE just move that function somewhere common and use it?)
Absolutely.  Didn't know that existed.  Would it make sense to put that directly on Value?  Or would you rather see it as a helper function in Local.h or someplace similar?
http://reviews.llvm.org/D9763
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list