[PATCH] D29011: [IR] Add Freeze instruction

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 03:50:07 PST 2017


deadalnix added a comment.

Hi,

Could you add a test case for the C API ? You can do so using the echo facility of llvm-c-test. See for instance: https://reviews.llvm.org/diffusion/L/browse/llvm/trunk/test/Bindings/llvm-c/atomics.ll



================
Comment at: include/llvm-c/Core.h:3028
+LLVMValueRef LLVMBuildFreeze(LLVMBuilderRef, LLVMValueRef Val,
+                                  const char *Name);
 
----------------
nit: please align the Name parameter properly.


https://reviews.llvm.org/D29011





More information about the llvm-commits mailing list