[PATCH] D83185: [Attributor] Introduce Attribute seed allow list.

Kuter Dinel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 15:02:28 PDT 2020


kuter added a comment.

In D83185#2134275 <https://reviews.llvm.org/D83185#2134275>, @uenoku wrote:

> Either is fine but I think it is more natural to forbid an empty list.


Do you mean returning a error if a empty `--attributor-seed-allow-list` option is present ?
Currently the size of list is being used to tell if a list is present or not.
I think I can use  `getNumOccurrences()` to replace this behaviour .


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83185/new/

https://reviews.llvm.org/D83185





More information about the llvm-commits mailing list