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

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 23 12:59:05 PDT 2021


jdoerfert added a comment.

In D101030#2713117 <https://reviews.llvm.org/D101030#2713117>, @ABataev wrote:

> Looks good, but would be nice to check with sanitizers that there are no uses-after-delete.

There is no heap allocation anymore but I guess I could have messed up the stack stuff.
I don't have a sanitizer build handy but the buildbots will yell at me if I messed up, is that sufficient?


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