Currently, the import of LLVMIR fails if the program contains debug
intrinsics. The revision adds support to import debug intrinsics that
have no debug expression attached and drops all debug intrinsics with a
non-empty debug expression. It also moves the existing debug intrinsics
into the "intr" namespace by deriving from LLVM_IntrOp.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/include/mlir/Dialect/LLVMIR/LLVMIntrinsicOps.td | ||
---|---|---|
280 | I prefer putting a TODO since the LLVMIR with debug information I have seen so far contains plenty of intrinsics with debug expressions. A hard error would thus prevent use from importing it at all. I plan to rework the error handling in the import in a follow up commit and will try to think of a better mechanism here. A warning that information has been dropped would be nice. However, we currently drop so much information (all kinds of meta information) that this may produce too many warnings / errors. |
Can we error out here at all?
Otherwise, this should have a TODO.