As requested in D53877 post-commit review, update naming of UNOP_NEG to UNOP_FNEG.
Also add UnaryOperator to the LLVMIsA##name macro. There does not appear to be existing tests for the LLVMIsA* functions, so I'm not sure how to test it.
Paths
| Differential D68588
[Bitcode] Update naming of UNOP_NEG to UNOP_FNEG ClosedPublic Authored by cameron.mcinally on Oct 7 2019, 1:02 PM.
Details
Summary As requested in D53877 post-commit review, update naming of UNOP_NEG to UNOP_FNEG. Also add UnaryOperator to the LLVMIsA##name macro. There does not appear to be existing tests for the LLVMIsA* functions, so I'm not sure how to test it.
Diff Detail
Event TimelineComment Actions Thank you, LG!
This revision is now accepted and ready to land.Oct 7 2019, 1:26 PM Closed by commit rL373958: [Bitcode] Update naming of UNOP_NEG to UNOP_FNEG (authored by mcinally). · Explain WhyOct 7 2019, 10:19 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 223637 llvm/include/llvm-c/Core.h
llvm/include/llvm/Bitcode/LLVMBitCodes.h
llvm/lib/Bitcode/Reader/BitcodeReader.cpp
llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
|
Commit this separately? "llvm-c: there's Unary operator"
I don't know what else might be missing for C API though.