github-actions[bot] wrote:
@Chengjunp Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a
https://github.com/AlexMaclean closed
https://github.com/llvm/llvm-project/pull/97113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jlebar approved this pull request.
https://github.com/llvm/llvm-project/pull/97113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jlebar wrote:
> Which file should I modify?
Use `git grep` to find where the text from that section of the langref lives?
https://github.com/llvm/llvm-project/pull/97113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Chengjunp wrote:
> This needs to be documented in the langref in this section, right?
> https://llvm.org/docs/LangRef.html#supported-constraint-code-list
Oh, you are right. Thank you for pointing this out. Which file should I modify?
Is it `llvm/docs/LangRef.rst`? Thanks!
https://github.com/jlebar commented:
LGTM other than the previous comment.
https://github.com/llvm/llvm-project/pull/97113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jlebar requested changes to this pull request.
This needs to be documented in the langref in this section, right?
https://llvm.org/docs/LangRef.html#supported-constraint-code-list
https://github.com/llvm/llvm-project/pull/97113
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (Chengjunp)
Changes
This change supports the 128-bit operands for inline ptx asm, both input and
output.\
\
The major changes are:
- Tablegen:\
Define Int128Regs in NVPTXRegisterInfo.td. But this register does not set
as
llvmbot wrote:
@llvm/pr-subscribers-backend-nvptx
Author: None (Chengjunp)
Changes
This change supports the 128-bit operands for inline ptx asm, both input and
output.\
\
The major changes are:
- Tablegen:\
Define Int128Regs in NVPTXRegisterInfo.td. But this register does not set
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you,
10 matches
Mail list logo