[Mlir-commits] [flang] [llvm] [mlir] [Flang][OpenMP][MLIR] Initial declare target to for variables implementation (PR #119589)

[Walter Lee]

googlewalt wrote:

Hi.  We are seeing that these tests fail when they are run with asan or ubsan enabled:

mlir/test/Target/LLVMIR/omptarget-data-use-dev-ordering.mlir
mlir/test/Target/LLVMIR/omptarget-nested-ptr-record-type-mapping-host.mlir
mlir/test/Target/LLVMIR/omptarget-nested-record-type-mapping-host.mlir
mlir/test/Target/LLVMIR/omptarget-nowait.mlir
mlir/test/Target/LLVMIR/omptarget-overlapping-record-member-map.mlir
mlir/test/Target/LLVMIR/omptarget-record-type-mapping-host.mlir
mlir/test/Target/LLVMIR/omptarget-record-type-with-ptr-member-host.mlir

In ~half the cases, sanitizers report issues (e.g. heap buffer overflow).  In ~half the cases sanitizer didn't complain but the tests fail.  Can you please take a look?



https://github.com/llvm/llvm-project/pull/119589