[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