[PATCH] D80991: [WIP][Attributor] AAPotentialValues Attribute

Shinji Okumura via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 28 10:16:07 PDT 2020


okura updated this revision to Diff 273949.
okura added a comment.
Herald added a reviewer: homerdin.
Herald added a reviewer: baziotis.

- change `PotentialValuesSet`
- handle division by zero in `calculateBinaryOperator`
- call `askSimplifiedValueFor` after value simplifying
- delete unnecessary debug prints
- add comments
- add test for PHINode

I add the threshold as a command-line option, but I got the following error message when I run it.
I don't know the cause and I got stuck. Could you tell me where my mistake is?

  : CommandLine Error: Option 'attributor-max-potential-values' registered more than once!
  LLVM ERROR: inconsistency in registered CommandLine options


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

https://reviews.llvm.org/D80991

Files:
  llvm/include/llvm/Transforms/IPO/Attributor.h
  llvm/lib/Transforms/IPO/Attributor.cpp
  llvm/lib/Transforms/IPO/AttributorAttributes.cpp
  llvm/test/Transforms/Attributor/potential.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80991.273949.patch
Type: text/x-patch
Size: 42266 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200628/31fc7684/attachment-0001.bin>


More information about the llvm-commits mailing list