Address Craig's comments.
Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Aug 12 2023
Jul 31 2023
Jul 30 2023
Check if div is legal for target. If it is legal, drop the transform.
Add nsw and nuw flags for add instruction.
Jun 19 2023
LGTM.
LGTM.
Jun 15 2023
Jun 13 2023
Is there any test case for the patch?
Jun 8 2023
LGTM
Jun 7 2023
Hi,
Is there anything else that I need to update?
LGTM.
Jun 6 2023
Address Simon's comments.
Address Phoebe's comments.
Rebase and update test cases.
Jun 5 2023
Rebase and update test case.
Jun 2 2023
LGTM for AMXINT8 change. Thanks, @MatzeB.
May 31 2023
May 30 2023
LGMT. I think it's a good start to have SPR schdule model in LLVM.
In D151537#4380763, @erichkeane wrote:I don't really see the justification here? Why do this change? If the intent is to just test a newer architecture, we can add tests for that, not change existing ones.
May 29 2023
LGTM
May 28 2023
May 26 2023
Rebase and add code comments.
NFC. Address Simon's comments.
May 25 2023
May 23 2023
May 19 2023
In D150526#4356290, @RKSimon wrote:Thanks @LuoYuanke - any more comments?
LGTM.
May 18 2023
May 14 2023
Address Phoebe and Simon's comments.
May 12 2023
It looks to me that initializing variable is a good practice to avoid security issue. If it doesn’t cause performance degradation, it should be better than uninitialized variable which may cause UB.
I once encountered issue that compiler behavior is not consistent when compiling several times, not sure it is related with uninitialized variable.
May 10 2023
May 8 2023
In D141485#4326543, @HaohaiWen wrote:@LuoYuanke , could you please help to verify avxvnni-combine.ll?
May 4 2023
Apr 28 2023
There is conflict when I try to revert the patch. I fixed the issue with rG40222ddcf8f5.
In D148980#4305430, @RKSimon wrote:@LuoYuanke I'm seeing tests failures on my local machine with expensive checks enabled - one buildbot is reporting something similar: https://lab.llvm.org/buildbot/#/builders/16/builds/47297
LGTM.
Apr 27 2023
The summary may need to describe the purpose or functionality for this patch.
In D149071#4303198, @RKSimon wrote:LGTM
@pengfei @LuoYuanke Are you happy with the performance vs code size trade off?
Address Phoebe's comments.
Apr 26 2023
ping
Apr 24 2023
Rebase
Update description
Address Craig's comments.
Apr 23 2023
Fix an opcode bug.
Update test case.
Add (avx512, 128, 256, 512) support.
Apr 22 2023
Add test case that there isn't the cross-loop dependency.
Rebase