[PATCH] D102136: [NewPM] Add C bindings for new pass manager

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/unittests/Passes/PassBuilderBindingsTest.cpp:19
+    LLVMInitializeNativeTarget();
+    const char *Triple = LLVMGetDefaultTargetTriple();
+    char *Err;
----------------
hctim wrote:
> ```
> /*===-- Triple ------------------------------------------------------------===*/
> /** Get a triple for the host machine as a string. The result needs to be
>   disposed with LLVMDisposeMessage. */
> char* LLVMGetDefaultTargetTriple(void);
> ```
> 
> Memory leak here caused by not calling `LLVMDisposeMessage(Triple)`.
fixed in ceb1ac9812cb0972cfa34cecc5e5704ae2a22063


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102136