RE: [PATCH v1 1/3] RISC-V: Allow RVV register overlap for vwmulsu.vv
"Li, Pan2" <[email protected]> Thu, 6 Aug 2026 00:30:51 +0000
| Newsgroups | gmane.comp.gcc.patches |
|---|---|
| Message-ID | <DS3PR11MB98442E8F178CF1315F15AEA3A9D22@DS3PR11MB9844.namprd11.prod.outlook.com> |
I see, thanks Robin. Only the test cases are LLM-assisted, but it will add that tag automatically when you ask help from LLM. I will make it clear before commit, as well as the underlying patches. Pan -----Original Message----- From: Robin Dapp <[email protected]> Sent: Wednesday, August 5, 2026 10:51 PM To: Li, Pan2 <[email protected]>; [email protected] Cc: [email protected]; [email protected]; [email protected]; [email protected]; Chen, Ken <[email protected]>; Liu, Hongtao <[email protected]> Subject: Re: [PATCH v1 1/3] RISC-V: Allow RVV register overlap for vwmulsu.vv > Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> I hope/guess you just default-added this to all of your messages? While assisted-by/co-authored-by would be OK for a testsuite change, it generally isn't for regular code. While this is of course a carbon copy of all the related changes, it's not a direct refactoring either and IMHO it's at least ambiguous. I'm not very happy having to play "copyright lawyer" now... but I think this is still fine. Legal insignificance is an exception to the general policy that LLM-assisted code is not OK. So this is OK but instead of just adding the tag, in the future please also state why you think the particular assisted-by change is legally insignificant. Otherwise we cannot ACK it. -- Regards Robin