[PATCH] D55192: [PowerPC] VSX register support for inline assembly

2018-12-07 Thread Zhang Kang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC348572: [PowerPC] VSX register support for inline assembly (authored by ZhangKang, committed by ). Herald added a subscriber: cfe-commits. Changed prior to commit: https://reviews.llvm.org/D55192?vs=177

[PATCH] D85599: [Clang] Consider __builtin_trap() and __builtin_debugtrap() as terminator

2020-08-09 Thread Zhang Kang via Phabricator via cfe-commits
ZhangKang created this revision. ZhangKang added reviewers: PowerPC, nemanjai, ahatanak, vitalybuka, dblaikie. ZhangKang added a project: LLVM. Herald added subscribers: Sanitizers, cfe-commits, steven.zhang, wuzish, dexonsmith, kristof.beyls. Herald added projects: clang, Sanitizers. ZhangKang re

[PATCH] D85599: [Clang] Consider __builtin_trap() and __builtin_debugtrap() as terminator

2020-08-10 Thread Zhang Kang via Phabricator via cfe-commits
ZhangKang updated this revision to Diff 284250. ZhangKang added a comment. Fix the case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85599/new/ https://reviews.llvm.org/D85599 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/test/CodeGen/builtin

[PATCH] D85599: [PowerPC] Remove isTerminator for TRAP instruction

2020-08-10 Thread Zhang Kang via Phabricator via cfe-commits
ZhangKang added a comment. In D85599#2206033 , @efriedma wrote: > Fixing a MachineVerifier issue by patching clang is generally wrong; if the > IR is valid, the backend should do something sane with it. If the IR isn't > valid, the IR Verifier should co

[PATCH] D85599: [Clang] Consider __builtin_trap() and __builtin_debugtrap() as terminator

2020-08-10 Thread Zhang Kang via Phabricator via cfe-commits
ZhangKang updated this revision to Diff 284424. ZhangKang added a comment. Herald added a subscriber: hiraditya. Remoeve isTerminator for TRAP instruction. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85599/new/ https://reviews.llvm.org/D85599 Fi