[llvm] [NVPTX] Fixing debug symbols for ptx target emitting (PR #101891)

Matt Davis via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 10 04:50:22 PDT 2024


enferex wrote:

> Hey, sorry I thought you said lint test. BTW, when you say add a lit test, do I need to create another test in the test directory for this change? Or just run the lit test and show if all the tests pass. Sorry I have never contributed to llvm and I do not know what the usual workflow is.

No problem.  Yes, I'd like to see a lit test.  This will help myself and others better understand the behavior of your change, as well as prevent regressions if this change is approved.  The LLVM docs ([link](https://llvm.org/docs/Contributing.html#how-to-submit-a-patch)) contain a guide for submitting patches. 

https://github.com/llvm/llvm-project/pull/101891


More information about the llvm-commits mailing list