[PATCH] D76958: [test-suite] Add support for Hexagon

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri marked an inline comment as done.
lebedev.ri added inline comments.


================
Comment at: MicroBenchmarks/libs/benchmark-1.3.0/src/cycleclock.h:85
   return (high << 32) | low;
+#elif defined(__hexagon__)
+  uint64_t  retv;
----------------
bcain wrote:
> lebedev.ri wrote:
> > bcain wrote:
> > > lebedev.ri wrote:
> > > > Can you please submit this as a pull request upstream?
> > > Oh, you bet!  I didn't know we could use PRs.
> > > 
> > > There's been a few threads and I've only skimmed them.  Now that you mention it I might've seen one that mentioned test-suite and PRs.
> > I'm not sure what you mean.
> > I was only talking about `MicroBenchmarks/libs/benchmark-1.3.0/*` change,
> > about upstreaming it to  https://github.com/google/benchmark
> Oh, whoops!  Jumped the gun there.
> 
> Ok, will do.  Once it lands there, do I make another review here to pull down the next release of that `benchmark`?  Or does that happen naturally somehow?
> Ok, will do. Once it lands there, do I make another review here to pull down the next release of that benchmark?

No, just cherry-pick that commit (note that there are at least 3 copies of `benchmark` within LLVM monorepo,
so you might want to patch them all at the same time..),
and add a comment about the cherry-pick into `benchmark/README.llvm`.



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

https://reviews.llvm.org/D76958