diff options
| author | Mehdi Amini <joker.eph@gmail.com> | 2021-11-01 19:15:39 +0000 |
|---|---|---|
| committer | Mehdi Amini <joker.eph@gmail.com> | 2021-11-02 22:35:47 +0000 |
| commit | ca0ed40e0000833e739f997f320bfafae35b4996 (patch) | |
| tree | 2145f18580d81e67b379a931400366521d6259ce /llvm/lib/Transforms/Utils/InlineFunction.cpp | |
| parent | 885072820c4ededc5ffa570ba8fd89b001fb9f11 (diff) | |
| download | llvm-ca0ed40e0000833e739f997f320bfafae35b4996.zip llvm-ca0ed40e0000833e739f997f320bfafae35b4996.tar.gz llvm-ca0ed40e0000833e739f997f320bfafae35b4996.tar.bz2 | |
Remove builder that takes SSA value instead of Attribute on ExtractValueOp, InsetValueOp, and InsertOnRangeOp
This builder exposed a somehow "unsafe" API: it pretends we can
construct an InsertOnRangeOp from a range of SSA values, even though
this will crash if these aren't the result of `arith.constant` since
the operation actually needs attribute values (a build method can't
fail gracefully).
That means that the caller must check for the producer, at which
point they can just assemble the attribute array directly and call
the existing builder.
The existing call-sites were even in a worse state here: they would
actually create a constant operation that wouldn't be used and only
serve to carry the attribute through the builder API.
Differential Revision: https://reviews.llvm.org/D112946
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
0 files changed, 0 insertions, 0 deletions