[PATCH] D108774: [OpenMP][FIX] Allow declare variant to work with reference types
Jon Chesterfield via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 26 13:19:42 PDT 2021
JonChesterfield added a comment.
That is a surprising root cause. It turns out std::cexp does in fact take the complex argument by const reference. The C versions take it by value. Shall we guard this with if (openmp) or similar to avoid it changing semantics for other languages?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108774/new/
https://reviews.llvm.org/D108774
More information about the cfe-commits
mailing list