[PATCH] D101030: [OpenMP] Overhaul `declare target` handling

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 23 09:43:12 PDT 2021


ABataev added a comment.

In D101030#2712738 <https://reviews.llvm.org/D101030#2712738>, @jdoerfert wrote:

> I'm testing a new version that does not allocate the structure on the heap. Instead we copy it. It is really infrequent and it should be <40 bytes when that happens, let's not over-optimize ;)

I don't worry about optimizations here, just about possible memory leak.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101030



More information about the llvm-commits mailing list