================

----------------
llvm-beanz wrote:

This seems like the wrong place to put this test. We don't need actual codegen 
here, we just need to make sure it parses.

I'd put this under SemaHLSL and instead use a run line like:
```hlsl
// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library %s -ast-dump | 
FileCheck %s
```

Restricting test cases to only running the parts of the compiler that are 
strictly required to verify correctness of a change reduces the time it takes 
to run the test. While in isolation for a single test that may only be shaving 
off fractions of a second, in aggregate across the entire test suite it can 
have a huge impact.

https://github.com/llvm/llvm-project/pull/126758
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to