- User Since
- Jul 21 2020, 2:19 PM (46 w, 3 d)
Fri, Jun 4
The tests you changed had 2 byte constants vs the 4 byte ones in the remaining f32 tests. But I don't know if that is significant.
Wed, May 26
I've found a difference between the if statement and foreach statement which seems to be related to casting. See https://bugs.llvm.org/show_bug.cgi?id=50486
Tue, May 25
Mon, May 24
Apr 29 2021
That is what you suggested last time on https://reviews.llvm.org/D99376
Apr 28 2021
Apr 16 2021
Apr 15 2021
Apr 6 2021
Mar 31 2021
Mar 30 2021
Mar 29 2021
Reverted since I don't fully understand the issue. It passed all tests on my system, but then upon rebasing top of tree it didn't. So I suspect the commutes may not be deterministic/ well constrained.
Confirmed that modifier operands are appropriately swapped if the
instruction is commuted, including opsel modifiers. Added at tests
update test using add_lshl
Mar 26 2021
remove sub as commutative and add add_lshl.
Its pretty bizarre to me that no testing cared if sub was commutative.
Mar 25 2021
It is unclear to me how to force a commute for testing purposes, but if anyone has an idea I can put that in. Vulkan short testlist passes fine with this patch. I will also test a compute psdb.
Mar 23 2021
Mar 22 2021
remove null initialization and fix clang-format
Mar 2 2021
Mar 1 2021
remove erroneous addition of OMod to cvt_flr_i32_f32 &
Feb 26 2021
Jan 22 2021
Jan 19 2021
I'm getting failing tests in check-llvm-mc-disassembler-amdgpu when applying this patch, can you take a look at that please?
Jan 12 2021
Jan 11 2021
Address format comments of foad.
Apply nfc edits suggested by dp
Jan 8 2021
Jan 5 2021
combined !con per comment by foad
Jan 4 2021
changed B_encoding predicate to be 10_3Insts
remove inst from gfx1030 only, instead of all gfx10
Nov 30 2020
Nov 4 2020
whitespace change for lint and rerun build
Nov 3 2020
Oct 21 2020
Oct 20 2020
remove accidental commented code
add OtherPredicates and blank line for rampitec
Oct 19 2020
I'm not sure exactly what you mean, but there is a workaround for branch instructions, that operates on the real instructions. Original fix at https://github.com/llvm/llvm-project/commit/9ab812d4752b2a1442426db2ccc17dc95d12eb04
Also please run PSDB for this change.
Oct 6 2020
Thank you for the review. I do not yet have commit access so please commit this on my behalf.
Oct 1 2020
I do not have commit access, please commit the patch for me.