aaron.ballman added a subscriber: bogner.
aaron.ballman added a comment.

In D159126#4625911 <https://reviews.llvm.org/D159126#4625911>, @cor3ntin wrote:

> @beanz The changes to TreeTransform.h cause a lot of HLSL test failures 
> similar to this one:
>
>   clang: 
> /home/cor3ntin/dev/compilers/LLVM/llvm-project/clang/lib/AST/ExprClassification.cpp:57:
>  Cl clang::Expr::ClassifyImpl(clang::ASTContext &, clang::SourceLocation *) 
> const: Assertion `isLValue()' failed.
>   PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ 
> and include the crash backtrace, preprocessed source, and associated run 
> script.
>   Stack dump:
>   0.      Program arguments: 
> /home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang -cc1 
> -internal-isystem 
> /home/cor3ntin/dev/compilers/LLVM/build-release/lib/clang/18/include 
> -nostdsysteminc -triple dxil-pc-shadermodel6.0-compute -emit-llvm -o - -O0 
> /home/cor3ntin/dev/compilers/LLVM/llvm-project/clang/test/CodeGenHLSL/builtins/RWBuffer-subscript.hlsl
>   1.      <eof> parser at end of file
>   2.      instantiating function definition 'hlsl::RWBuffer<int>::RWBuffer'
>    #0 0x00005562645f33c7 llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x3d663c7)
>    #1 0x00005562645f0f9e llvm::sys::RunSignalHandlers() 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x3d63f9e)
>    #2 0x00005562645f3bba SignalHandler(int) Signals.cpp:0:0
>    #3 0x00007fc625c3c4b0 (/lib/x86_64-linux-gnu/libc.so.6+0x3c4b0)
>    #4 0x00007fc625c90ffb __pthread_kill_implementation 
> ./nptl/pthread_kill.c:44:76
>    #5 0x00007fc625c90ffb __pthread_kill_internal ./nptl/pthread_kill.c:78:10
>    #6 0x00007fc625c90ffb pthread_kill ./nptl/pthread_kill.c:89:10
>    #7 0x00007fc625c3c406 raise ./signal/../sysdeps/posix/raise.c:27:6
>    #8 0x00007fc625c2287c abort ./stdlib/abort.c:81:7
>    #9 0x00007fc625c2279b _nl_load_domain ./intl/loadmsgcat.c:1177:9
>   #10 0x00007fc625c33b86 (/lib/x86_64-linux-gnu/libc.so.6+0x33b86)
>   #11 0x0000556267a3f1a9 clang::Expr::ClassifyImpl(clang::ASTContext&, 
> clang::SourceLocation*) const 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x71b21a9)
>   #12 0x0000556267a3fa37 clang::Expr::isModifiableLvalue(clang::ASTContext&, 
> clang::SourceLocation*) const 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x71b2a37)
>   #13 0x0000556266ffbd1a CheckForModifiableLvalue(clang::Expr*, 
> clang::SourceLocation, clang::Sema&) SemaExpr.cpp:0:0
>   #14 0x0000556266ffb4b4 clang::Sema::CheckAssignmentOperands(clang::Expr*, 
> clang::ActionResult<clang::Expr*, true>&, clang::SourceLocation, 
> clang::QualType, clang::BinaryOperatorKind) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x676e4b4)
>   #15 0x0000556266fddd2e 
> clang::Sema::CreateBuiltinBinOp(clang::SourceLocation, 
> clang::BinaryOperatorKind, clang::Expr*, clang::Expr*) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x6750d2e)
>   #16 0x0000556267572779 clang::TreeTransform<(anonymous 
> namespace)::TemplateInstantiator>::TransformBinaryOperator(clang::BinaryOperator*)
>  SemaTemplateInstantiate.cpp:0:0
>   #17 0x000055626756ceb5 clang::TreeTransform<(anonymous 
> namespace)::TemplateInstantiator>::TransformStmt(clang::Stmt*, 
> clang::TreeTransform<(anonymous 
> namespace)::TemplateInstantiator>::StmtDiscardKind) 
> SemaTemplateInstantiate.cpp:0:0
>   #18 0x000055626758c4ba clang::TreeTransform<(anonymous 
> namespace)::TemplateInstantiator>::TransformCompoundStmt(clang::CompoundStmt*,
>  bool) SemaTemplateInstantiate.cpp:0:0
>   #19 0x000055626756ce43 clang::Sema::SubstStmt(clang::Stmt*, 
> clang::MultiLevelTemplateArgumentList const&) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x6cdfe43)
>   #20 0x00005562675c6bab 
> clang::Sema::InstantiateFunctionDefinition(clang::SourceLocation, 
> clang::FunctionDecl*, bool, bool, bool) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x6d39bab)
>   #21 0x00005562675c9cbe clang::Sema::PerformPendingInstantiations(bool) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x6d3ccbe)
>   #22 0x0000556266bf08a5 
> clang::Sema::ActOnEndOfTranslationUnitFragment(clang::Sema::TUFragmentKind) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x63638a5)
>   #23 0x0000556266bf1028 clang::Sema::ActOnEndOfTranslationUnit() 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x6364028)
>   #24 0x0000556266ab548c 
> clang::Parser::ParseTopLevelDecl(clang::OpaquePtr<clang::DeclGroupRef>&, 
> clang::Sema::ModuleImportState&) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x622848c)
>   #25 0x0000556266ab007e clang::ParseAST(clang::Sema&, bool, bool) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x622307e)
>   #26 0x000055626519ac60 clang::FrontendAction::Execute() 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x490dc60)
>   #27 0x000055626510c96f 
> clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x487f96f)
>   #28 0x000055626527cc5b 
> clang::ExecuteCompilerInvocation(clang::CompilerInstance*) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x49efc5b)
>   #29 0x000055626341cf64 cc1_main(llvm::ArrayRef<char const*>, char const*, 
> void*) (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x2b8ff64)
>   #30 0x0000556263419e3d ExecuteCC1Tool(llvm::SmallVectorImpl<char const*>&, 
> llvm::ToolContext const&) driver.cpp:0:0
>   #31 0x0000556263418b9e clang_main(int, char**, llvm::ToolContext const&) 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x2b8bb9e)
>   #32 0x00005562634298d1 main 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x2b9c8d1)
>   #33 0x00007fc625c23a90 __libc_start_call_main 
> ./csu/../sysdeps/nptl/libc_start_call_main.h:74:3
>   #34 0x00007fc625c23b49 call_init ./csu/../csu/libc-start.c:128:20
>   #35 0x00007fc625c23b49 __libc_start_main ./csu/../csu/libc-start.c:347:5
>   #36 0x0000556263416325 _start 
> (/home/cor3ntin/dev/compilers/LLVM/build-release/bin/clang+0x2b89325)
>   FileCheck error: '<stdin>' is empty.
>   FileCheck command line:  
> /home/cor3ntin/dev/compilers/LLVM/build-release/bin/FileCheck 
> /home/cor3ntin/dev/compilers/LLVM/llvm-project/clang/test/CodeGenHLSL/builtins/RWBuffer-subscript.hlsl
>
> any idea would be appreciated! Thanks

CC @bogner as well


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D159126/new/

https://reviews.llvm.org/D159126

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to