[PATCH] D142839: [AMDGPU] Avoid using tuple where pair does suffice

Xiaoqiu Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 09:54:24 PDT 2023


xiaoqili added a comment.

Same issue happened on Android prebuilt clang 8 in NDK20. Seem it's having a stricter rule for deduction the template arguments.
Root cause should be the different definition of AssemblerDirectiveBegin/AssemblerDirectiveEnd.
It's defined as "const char * ......"in std::tie which is the left value.
But the right value which in HSAMD::V3 and HSAMD:: it's defined as "constexpr char ......[]".
Tried to make it defined as "const char * ......" in HSAMD::V3 and HSAMD in local, it can pass for this error(just a test I don't think it's the right fix).


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

https://reviews.llvm.org/D142839



More information about the llvm-commits mailing list