This patch supplements missing test case for D107353.
- Fix wrong descriptions in 64-bit mode test case
- Added testcase under 32-bit mode
Paths
| Differential D108507
[NFC][LoopIdiom] Add more test case to runtime-determined memset size ClosedPublic Authored by eopXD on Aug 21 2021, 3:10 AM.
Details Summary This patch supplements missing test case for D107353.
Diff Detail
Event TimelineComment Actions
As @bmahjour was commenting in D108112, he questioned on how the patch will operate on 32-bit mode.
eopXD marked an inline comment as done. Comment ActionsAdjust tescases to show its relationship with D108112.
There was only 64-bit test cases when I was doing D108112.
Yes. D108112 should modify upon the test case this patch provides.
Comment Actions Address comments.
This revision is now accepted and ready to land.Oct 20 2021, 12:41 PM Closed by commit rG76db6d80805a: [NFC][LoopIdiom] Add more test case to runtime-determined memset size (authored by eopXD). · Explain WhyOct 21 2021, 12:05 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 381156 llvm/test/Transforms/LoopIdiom/memset-runtime-32bit.ll
llvm/test/Transforms/LoopIdiom/memset-runtime-64bit.ll
llvm/test/Transforms/LoopIdiom/memset-runtime.ll
|
match the order of arguments with the actual function.