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

Francesco Petrogalli via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 11 13:55:11 PST 2018


fpetrogalli added a comment.

In D54412#1327574 <https://reviews.llvm.org/D54412#1327574>, @rengolin wrote:

> In D54412#1327560 <https://reviews.llvm.org/D54412#1327560>, @fpetrogalli wrote:
>
> > Is there a document that explains how to submit an RFC?
>
>
> Hi Francesco,
>
> It's not that complex for a document. The text you have here would be more than fine, and the discussion that happened is exactly what we look for, but Phab reviews only copy llvm-commits, not llvm-dev, and that is the main problem.


I see! Now it is clear to me - I thought people were asking me to create a new review on phabricator of the same document, and move the discussion there. What was really being asked was to submit the actual text of the proposal in the body of the email announcing the RFC... I will do it.

> Since there's a lot of high quality discussions here, I suggest you post the same content, but with a summary of the feedback you had, and the changes in design you had to do because of that.
> 
> This would avoid Hal and Hideki from having to say it all over again, and will get more people into the right point in the conversation.

I would like to avoid writing a summary, and take care of clarifying what is being asked by pointing them at specific comments in the review if the question has already been answered in phabricator.

> cheers,
> --renato




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

https://reviews.llvm.org/D54412





More information about the llvm-commits mailing list