joker-eph wrote:
PR should be named according to what they are actually achieving, I'm not sure
why the GitHub issue title is relevant?
https://github.com/llvm/llvm-project/pull/90391
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://l
luolent wrote:
> Please update the PR subject as its a lot more than just X86AsmParser.cpp
Hi @RKSimon
All the issues mentioned are fixed. The tile of the PR is misleading.
The title is the same as the issue (#85868) it corresponds to.
Got a look to other PR's and I thought that this is the u
https://github.com/luolent updated
https://github.com/llvm/llvm-project/pull/90391
>From 54c6c6b7d71f5ff293412f5f91e9f880480284f8 Mon Sep 17 00:00:00 2001
From: luolent
Date: Sat, 27 Apr 2024 22:17:58 +0300
Subject: [PATCH] llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp:3804: lacking
() for cl
https://github.com/luolent updated
https://github.com/llvm/llvm-project/pull/90391
>From 2a6b6a3b160ae49b20da6c8cc9f8d858ddb598eb Mon Sep 17 00:00:00 2001
From: luolent
Date: Sat, 27 Apr 2024 22:17:58 +0300
Subject: [PATCH] llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp:3804: lacking
() for cl
https://github.com/RKSimon commented:
Please address the clang-format warnings the CI has reported
https://github.com/llvm/llvm-project/pull/90391
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 738c135ee09de66a26805530d314c2f28d1eca02
f860432b9434c81ab55d5eded9c976edc58c3fab --
https://github.com/RKSimon commented:
Please update the PR subject as its a lot more than just X86AsmParser.cpp
https://github.com/llvm/llvm-project/pull/90391
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
llvmbot wrote:
@llvm/pr-subscribers-backend-systemz
@llvm/pr-subscribers-backend-amdgpu
@llvm/pr-subscribers-clang
Author: None (luolent)
Changes
Hi,
parenthesis were added as requested on ternary operators with non trivial
conditions.
I used this [precedence
table](https://en.cpprefere
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, it
https://github.com/luolent created
https://github.com/llvm/llvm-project/pull/90391
Hi,
parenthesis were added as requested on ternary operators with non trivial
conditions.
I used this [precedence
table](https://en.cppreference.com/w/cpp/language/operator_precedence) to make
sure we get the
10 matches
Mail list logo
