Arghnews wrote:

> We're putting these tests in the appropriate 
> llvm-project\clang\test\CodeGen\X86*-builtins.c files - search for 
> TEST_CONSTEXPR

I have been splitting these out as specified, however I've run into an issue.

Test files using `-fexperimental-new-constant-interpreter` like 
`clang/test/CodeGen/X86/avx2-builtins.c` or `avx512f-builtins.c`, that I'd put 
tests in, error when using the experimental constant interpreter flag.

If I remove this flag, they work fine (with a little fixing/neatening things up 
for casts etc.)

I guess this is because `clang/lib/AST/ExprConstant` is the current interpreter 
and we're not adding support to this new one.

I can't see a way to opt out of using this for specific lines when doing 
`TEST_CONSTEXPR`. Shall I just make new files for these cases? Is adding 
support for these in the new interpreter a large task, beyond the scope of this 
etc.?

Please suggest what you think is the best way forward here, thanks
@RKSimon 

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

Reply via email to