This change is triggered by D109343. where clang-tidy was not satisfied with existing function name.
Names are changed as per clang formatting.
Paths
| Differential D109590
[NFCI] Rename functions LLVMDIBuilder* to llvmDIBuilder* AbandonedPublic Authored by alok on Sep 10 2021, 3:16 AM.
Details
Diff Detail
Unit TestsFailed
Event TimelineComment Actions The rest of the C API uses an "LLVM" prefix. Does it really make sense to use a different one for the DIBuilder functions? Comment Actions
Thanks for your comment. Do you vote for changing all the functions or leaving all the functions as it is ? This revision now requires changes to proceed.Sep 10 2021, 6:09 AM Comment Actions
Thanks. I got it. Those clang-tidy warning should be ignored or clang-tidy should be fixed to not warn in such cases.
Revision Contents
Diff 371851 llvm/include/llvm-c/DebugInfo.h
llvm/lib/IR/DebugInfo.cpp
llvm/tools/llvm-c-test/debuginfo.c
|