[PATCH] D54412: [RFC] Re-implementing -fveclib with OpenMP

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 10 08:26:05 PST 2018


lebedev.ri added a comment.

In D54412#1325449 <https://reviews.llvm.org/D54412#1325449>, @fpetrogalli wrote:

> In D54412#1317294 <https://reviews.llvm.org/D54412#1317294>, @lebedev.ri wrote:
>
> > I feel like pointing out that this RFC did *NOT* go to the lists, which makes zero sense, since it's RFC.
>
>
> Hi - sorry about that - I thought that sending it out on llvm-dev and cfe-dev was enough.


No.

>> Usually, it is recommended to close the old review and submit a new one, making sure that it *does* go to the lists..
> 
> I see. I am happy to create a new one if that is a requirement for the RFC submission. One question:



> how do I make sure it goes to the lists? I thought I did it and I want to avoid making the same error.

By setting correct "repo" field (best) and/or adding the appropriate -commits list as the subscriber.

> Kind regards,
> 
> Francesco




CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54412/new/

https://reviews.llvm.org/D54412





More information about the llvm-commits mailing list