[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-22 Thread Pengfei Wang via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe8bce8399631: [X86] Enable compilation of user interrupt handlers. (authored by pengfei). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-12 Thread LuoYuanke via Phabricator via cfe-commits
LuoYuanke accepted this revision. LuoYuanke added a comment. This revision is now accepted and ready to land. LGTM. But wait one or two days to see if there is more comments from Craig and HJ. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99708/new/ https://reviews.llvm.org/D99708 ___

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-11 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing marked an inline comment as done. tianqing added a comment. Does anyone has further comments? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99708/new/ https://reviews.llvm.org/D99708 ___ cfe-commits mailing list cfe-commits@lists.llv

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-07 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing updated this revision to Diff 335980. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99708/new/ https://reviews.llvm.org/D99708 Files: clang/lib/Headers/uintrintrin.h llvm/lib/Target/X86/X86ExpandPseudo.cpp llvm/test/CodeGen/X86/x86-64-intrcc-uintr.ll Index: llvm/test/Code

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-06 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added inline comments. Comment at: llvm/test/CodeGen/X86/x86-64-intrcc-uintr.ll:1 +; RUN: llc < %s | FileCheck %s -check-prefixes=CHECK,CHECK-USER +; RUN: llc -O0 < %s | FileCheck %s -check-prefixes=CHECK0,CHECK0-USER Please use update_llc_test_check

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-06 Thread LuoYuanke via Phabricator via cfe-commits
LuoYuanke added a comment. LGMT. Thank you! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99708/new/ https://reviews.llvm.org/D99708 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-06 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing updated this revision to Diff 335426. tianqing added a comment. Update handling of -mcmodel=kernel. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99708/new/ https://reviews.llvm.org/D99708 Files: clang/lib/Headers/uintrintrin.h llvm/lib/Target/X86/X86ExpandPseudo.cpp llv

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread H.J Lu via Phabricator via cfe-commits
hjl.tools added a comment. In D99708#2664372 , @craig.topper wrote: > In D99708#2664351 , @hjl.tools wrote: > >> In D99708#2664218 , @craig.topper >> wrote: >> >>> In D9970

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D99708#2664351 , @hjl.tools wrote: > In D99708#2664218 , @craig.topper > wrote: > >> In D99708#2664164 , @hjl.tools >> wrote: >> >>> In D99

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread H.J Lu via Phabricator via cfe-commits
hjl.tools added a comment. In D99708#2664218 , @craig.topper wrote: > In D99708#2664164 , @hjl.tools wrote: > >> In D99708#2664076 , @LuoYuanke >> wrote: >> >>> In D99708#2

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D99708#2664164 , @hjl.tools wrote: > In D99708#2664076 , @LuoYuanke wrote: > >> In D99708#2663989 , @craig.topper >> wrote: >> >>> A user in

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread H.J Lu via Phabricator via cfe-commits
hjl.tools added a comment. In D99708#2664076 , @LuoYuanke wrote: > In D99708#2663989 , @craig.topper > wrote: > >> A user interrupt is different than a regular interrupt right? It doesn't >> make sense that we wou

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread LuoYuanke via Phabricator via cfe-commits
LuoYuanke added a comment. In D99708#2663989 , @craig.topper wrote: > A user interrupt is different than a regular interrupt right? It doesn't make > sense that we would change the behavior of the interrupt calling convention > just because the the user

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. A user interrupt is different than a regular interrupt right? It doesn't make sense that we would change the behavior of the interrupt calling convention just because the the user interrupt instructions are enabled. That would occur just from passing a -march for a

[PATCH] D99708: [X86] Enable compilation of user interrupt handlers.

2021-04-01 Thread Wang Tianqing via Phabricator via cfe-commits
tianqing created this revision. tianqing added reviewers: pengfei, LuoYuanke, craig.topper. Herald added a subscriber: hiraditya. tianqing requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Add __uintr_frame structure and use