[clang] [llvm] [DirectX] add Function name to DiagnosticInfoUnsupported Msg in DXILOpLowering (PR #136234)
Justin Bogner via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 18 10:04:59 PDT 2025
bogner wrote:
This doesn't seem right. Are we just misusing `DiagnosticInfoUnsupported` here? The `Function` argument for `DiagnosticInfoWithLocationBase` is clearly meant to be part of the location (as evidenced by the "in function XYZ" part of the message). I suspect that we should really be changing the unknown intrinsic diagnostic to use `DiagnosticInfoGeneric` rather than `DiagnosticInfoUnsupported`, and we should probably update the other use of `DiagnosticInfoUnsupported` in `OpLowerer::replaceFunction` to use `DiagnosticInfoGenericWithLoc` and pass in the parent function of the `CallInst` rather than the callee.
https://github.com/llvm/llvm-project/pull/136234
More information about the cfe-commits
mailing list