Tue, Aug 13
Tue, Aug 6
Jul 4 2019
Jun 24 2019
Possibly, but our majority use case is to not need the disassembly, and our thinking is that we don't want to change the tool flow between the majority case and the minority case. So can we get this fix in please?
I don't think anyone is a fan of dumpcode. But we're still in the position that the proper disassembler does not support gfx6 or gfx7, and we need to get this particular problem fixed in the short term.
Jun 19 2019
For this bug, whatever we do with a mir test, it is not going to be reliable in failing if the bug is present. Maybe there is a unit testing framework for LiveRangeCalc tests that I could add a test to.
Jun 18 2019
Jun 13 2019
Jun 12 2019
V2: Lit test fix.
V2: Addressed review comments re test.
Jun 11 2019
May 30 2019
V2: Target-specific attribute instead of target feature, as suggested by Stas.
May 29 2019
May 21 2019
LGTM, but I don't think I know the legalization code well enough to approve this.
May 16 2019
Is someone now able to approve this? Eli?
May 14 2019
V4: update_mir_test_checks the test.
May 8 2019
Ping: Could someone approve this please? Thanks.
Hi Samuel. Sorry for the delay; I kind of lost track of this change.
Apr 17 2019
V3: Further reduced test case.
V2: Addressed review comments.
V2: Addressed review comment: moved tests.
Apr 15 2019
Bug for adding remaining test cases is https://bugs.llvm.org/show_bug.cgi?id=41500
V3: Addressed review comments.
PS I'm only speaking for LLPC. I don't know if Mesa uses it.
I think the only obstacle is getting round to doing the LLPC changes. So this is hopefully a short term fix until we can get around to it.
I have cut down the test a bit more and put it into fneg-combines.ll. I did not manage to repro any problems with the other cases that I added fixes for.
V2: Cut down test a bit more and put it in fneg-combines.ll.
Apr 13 2019
I did not add a test because I believe that we do not currently set the source modifiers in codegen, so this code never gets exercised.
Apr 12 2019
That's the best bugpoint could do with the test.
Apr 9 2019
Hopefully fixed by D60457.
Apr 1 2019
Mar 29 2019
LGTM modulo the wrong license on the new file.
Mar 27 2019
OK thanks Matt.