pengfei added inline comments.
================ Comment at: clang/lib/Sema/SemaStmtAsm.cpp:679 + !llvm::isPowerOf2_32(OutSize)) + targetDiag(OutputExpr->getExprLoc(), diag::err_store_value_to_reg); + ---------------- jyu2 wrote: > Error message is not very clear to me. I think we should create more > specified error message there. Like power of two, or size < 8 or > pointer > size? > > Using error message selector. Use `getIntTypeForBitwidth` instead. So we don't need to check for each case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107141/new/ https://reviews.llvm.org/D107141 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits