[PATCH] D115857: [LoongArch 1/6] Add triples loongarch{32, 64} for the upcoming LoongArch target
Lu Weining via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 10 03:40:26 PST 2022
SixWeining added a comment.
In D115857#3310452 <https://reviews.llvm.org/D115857#3310452>, @rengolin wrote:
> Than you @SixWeining, the 6 patches have been merged now.
>
> Be on the lookout for buildbot breakages (you'll get emails), but on a clean build on x86_64 + check-all I have seen no breakages.
>
> Also thanks to @xen0n @myhsu for the reviews.
>
> From now on, patches can come at any time, just make sure to announce the dependencies on each review, if they have one.
>
> Feel free to continue to copy the people that helped you in the initial submission, and see if you can help other new and experimental targets as well (ARC, CSKY, M68k, MSP430, SPIR-V), with your gained experience in this process. The more you review other people's patches, the more likely they'll review yours.
>
> Finally, welcome to the LLVM family!
@rengolin @xen0n @myhsu Thanks for your help to make it happen. We will try our best to review other people's patches. We believe that goodwill is our currency <https://llvm.org/docs/CodeReview.html#code-reviews-speed-and-reciprocity>.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115857/new/
https://reviews.llvm.org/D115857
More information about the llvm-commits
mailing list