[llvm] [DirectX] Set whole-module flags prior to evaluating per-function flags (PR #139967)

Deric C. via llvm-commits llvm-commits at lists.llvm.org
Thu May 15 10:46:46 PDT 2025


Icohedron wrote:

> I think the bugs you are fixing were mostly found because of the validator? Should we be doing some testing that invokes the validator something similaor to what `llvm/test/tools/dxil-dis/` tests do?

Yes, the bugs were found in the validator. 
Unlike dxil-dis, dxv (the validator) isn't included as a tool with LLVM. Perhaps dxv should be made available as llvm-dxv perhaps?

Or, the DirectX tests could have RUN lines conditioned on having dxv in the path, and runs the validator if it is in the path; similar to spirv-val I saw in SPIRV tests.

Either way, I think this topic should be relegated to a separate issue and PR.

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


More information about the llvm-commits mailing list