[PATCH] D90103: Add OpenMP for optimization

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 23 02:11:17 PST 2020


sstefan1 added a comment.

In D90103#2410284 <https://reviews.llvm.org/D90103#2410284>, @abidmalikwaterloo wrote:

> The tests should be in llvm-project/llvm/test/Transforms/OpenMP.

Yes, no need for new folder.



================
Comment at: .arcconfig:2
 {
-  "phabricator.uri" : "https://reviews.llvm.org/",
-  "repository.callsign" : "G",
-  "conduit_uri" : "https://reviews.llvm.org/"
+  "phabricator.uri" : "https://reviews.llvm.org/"
 }
----------------
abidmalikwaterloo wrote:
> sstefan1 wrote:
> > This seems unrelated.
> Should I then include the following?
> 
> "repository.callsign" : "G",
> "conduit_uri" : "https://reviews.llvm.org/"
I'm saying I don't see the reason for this to be in the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90103



More information about the llvm-commits mailing list