[all-commits] [llvm/llvm-project] 2fad6e: [InlineAsm] wrap Kind in enum class NFC
Nick Desaulniers via All-commits
all-commits at lists.llvm.org
Thu Aug 31 08:55:12 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 2fad6e69851ecbaf01553cd5790e4575290bf0a9
https://github.com/llvm/llvm-project/commit/2fad6e69851ecbaf01553cd5790e4575290bf0a9
Author: Nick Desaulniers <ndesaulniers at google.com>
Date: 2023-08-31 (Thu, 31 Aug 2023)
Changed paths:
M llvm/include/llvm/IR/InlineAsm.h
M llvm/lib/CodeGen/AsmPrinter/AsmPrinterInlineAsm.cpp
M llvm/lib/CodeGen/GlobalISel/InlineAsmLowering.cpp
M llvm/lib/CodeGen/MachineInstr.cpp
M llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp
M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
M llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.h
M llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
M llvm/lib/CodeGen/TargetInstrInfo.cpp
M llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp
M llvm/lib/Target/AVR/AVRISelLowering.cpp
M llvm/lib/Target/CSKY/CSKYISelDAGToDAG.cpp
M llvm/lib/Target/Hexagon/HexagonISelLowering.cpp
M llvm/lib/Target/PowerPC/PPCISelLowering.cpp
M llvm/lib/Target/Sparc/SparcISelDAGToDAG.cpp
M llvm/lib/Target/X86/X86FloatingPoint.cpp
Log Message:
-----------
[InlineAsm] wrap Kind in enum class NFC
Should add some minor type safety to the use of this information, since
there's quite a bit of metadata being laundered through an `unsigned`.
I'm looking to potentially add more bitfields to that `unsigned`, but I
find InlineAsm's big ol' bag of enum values and usage of `unsigned`
confusing, type-unsafe, and un-ergonomic. These can probably be better
abstracted.
I think the lack of static_cast outside of InlineAsm indicates the prior
code smell fixed here.
Reviewed By: qcolombet
Differential Revision: https://reviews.llvm.org/D159242
More information about the All-commits
mailing list