Commit 73e5c12
committed
[IR] Allow llvm.experimental.memset.pattern to take any sized type as the pattern argument
I initially thought starting with a more narrow definition and later
expanding would make more sense. But as pointed out in review for
PR #129220, this restriction is generating additional unnecessary work.
This patch alters the intrinsic to accept patterns of any type. Future
patches will update LoopIdiomRecognize and PreISelIntrinsicLowering to
take advantage of this. The verifier will complain if an unsized type is
used. I've additionally taken the opportunity to remove a comment from
the LangRef about some bit widths potentially not being supported by the
target. I don't think this is any more true than it is for arbitrary
width loads/stores which don't carry a similar warning that I can see.1 parent 5b6b4fd commit 73e5c12
File tree
4 files changed
+17
-3
lines changed- llvm
- docs
- include/llvm/IR
- lib/IR
- test/Verifier
4 files changed
+17
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15640 | 15640 | | |
15641 | 15641 | | |
15642 | 15642 | | |
15643 | | - | |
15644 | | - | |
| 15643 | + | |
| 15644 | + | |
15645 | 15645 | | |
15646 | 15646 | | |
15647 | 15647 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1025 | 1025 | | |
1026 | 1026 | | |
1027 | 1027 | | |
1028 | | - | |
| 1028 | + | |
1029 | 1029 | | |
1030 | 1030 | | |
1031 | 1031 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5583 | 5583 | | |
5584 | 5584 | | |
5585 | 5585 | | |
| 5586 | + | |
5586 | 5587 | | |
| 5588 | + | |
| 5589 | + | |
| 5590 | + | |
5587 | 5591 | | |
5588 | 5592 | | |
5589 | 5593 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
0 commit comments