[Lldb-commits] [PATCH] D98879: [lldb/PlatformPOSIX] Change LoadImage default to RTLD_LAZY

[Vedant Kumar via Phabricator via lldb-commits]

vsk added a comment.

The best way I can think of to test this is to:

- Make a library, T1, out of:

  % cat t1.c
  extern void use();
  void f1() {}
  void f2() { use(); }
  - Make a library, T2, out of:

% cat t2.c
void use() {}

  - Link T1.dylib against T2.dylib, so that the reference to `use` isn't undefined.
  - Now, rebuild T2.dylib using an empty source file (delete the definition of `use`).
  - Launch a process, then do "process load T1.dylib"; this will only succeed under RTLD_LAZY.
  
  Any concerns?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98879