pengfei accepted this revision. pengfei added a comment. This revision is now accepted and ready to land.
LGTM. ================ Comment at: clang/lib/CodeGen/CGStmt.cpp:2491 + std::string::size_type position1 = + Constraints.find("={" + Clobber.str() + "}"); + if (position1 != std::string::npos) { ---------------- I'm not sure if there's real case that there's `"={edx}"` for MS inline asm, but there's no problem we handle it here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94466/new/ https://reviews.llvm.org/D94466 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits