[PATCH] D71830: [OpenMP][Part 2] Use reusable OpenMP context/traits handling

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 6 09:08:58 PST 2020


jdoerfert marked an inline comment as done.
jdoerfert added inline comments.


================
Comment at: clang/include/clang/Serialization/ASTRecordReader.h:272
+
+  template <> OpenMPTraitInfo *readUserType() { return readOpenMPTraitInfo(); }
+
----------------
kiranchandramohan wrote:
> jdoerfert wrote:
> > kiranchandramohan wrote:
> > > Compiler throws up this error.
> > > error: explicit specialization in non-namespace scope ‘class clang::ASTRecordReader’
> > Oh, my compiler was happy. Let me rebase and see what the pre-merge bots say so I might get some insight into the problem.
> Were you able to reproduce the error? I was using gcc 9.2 compiler.
I have not seen the error yet. I build with clang. Do you happen to have an idea how to refactor this? I will look into it with a new gcc asap.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71830





More information about the cfe-commits mailing list