[PATCH] D83185: [Attributor] Introduce Attribute seed allow list.
    Stefan Stipanovic via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul  6 12:53:29 PDT 2020
    
    
  
sstefan1 added a comment.
Herald added a subscriber: bbn.
Can you add a test using this option?
================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:1460
+    return true;
+  return std::count(SeedAllowList.begin(), SeedAllowList.end(), AA.getName());
+}
----------------
would it make sense to make this always check lower case names, to avoid mistakes?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83185/new/
https://reviews.llvm.org/D83185
    
    
More information about the llvm-commits
mailing list