Christophe Lyon writes: > On 30 June 2015 at 11:12, Jiong Wang <jiong.w...@arm.com> wrote: >> >> Christophe Lyon writes: >> >>> Hi Jiong, >>> >>> It seems to me that you committed patches 1/N, 2/N and 3/N as only 1 >>> commit (rev 225121). >>> >>> Am I right? >> >> Yes, 1/2/3 are quite tiny, and seperated for easy review. While I >> committed them as one patch. > > > Thanks for the clarification, I don't think this is standard practice.
OK. It's better to add at least links to the patch review at gcc-patches archive so that people can find the background discussion easily. I was following https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00254.html. > >>> >>> >>> On 26 June 2015 at 16:39, Marcus Shawcroft <marcus.shawcr...@gmail.com> >>> wrote: >>>> On 20 May 2015 at 12:21, Jiong Wang <jiong.w...@arm.com> wrote: >>>>> >>>>> Add new unspec name UNSPEC_TLSLE, use it for all tlsle pattern. >>>>> >>>>> ok for trunk? >>>>> >>>>> 2015-05-19 Jiong Wang <jiong.w...@arm.com> >>>>> >>>>> gcc/ >>>>> * config/aarch64/aarch64.md (UNSPEC_TLSLE): New enumeration. >>>>> (tlsle): Use new unspec name. >>>>> (tlsle_<mode>): Ditto. >>>> >>>> OK /Marcus >> >> -- >> Regards, >> Jiong >> -- Regards, Jiong
