- User Since
- Jan 26 2017, 7:07 AM (116 w, 6 d)
Wed, Apr 17
V3: Further reduced test case.
V2: Addressed review comments.
V2: Addressed review comment: moved tests.
Mon, Apr 15
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.
Sat, Apr 13
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.
Fri, Apr 12
That's the best bugpoint could do with the test.
Tue, Apr 9
Hopefully fixed by D60457.
Mon, Apr 1
Fri, Mar 29
LGTM modulo the wrong license on the new file.
Wed, Mar 27
OK thanks Matt.
Mar 22 2019
Mar 21 2019
Mar 20 2019
Thanks for the better test Matt. But I'll abandon this one in favor of Michael's improved fix D59608.
Thanks Michael; that is much better than my fix in D59556. Let's go with this one and I'll abandon my one.
The test is already reduced as much as I can. Removing anything in there makes the problem disappear. Constructing a new test case using llvm.uadd.with.overflow does not show the problem. Can we go with this test case?
V4: Generate vendor=AMDGPU, type=32 for the metadata note record, as now expected by PAL.
Mar 19 2019
V3: Rebased, and addressed "single char in single quotes" review comment from the other change.
V3: Fixed review comments.
Mar 18 2019
V2: Like the previous commit, no longer supports a method for LLPC to
call to write the PAL metadata into IR metadata. The plan now is that LLPC will use MsgPackDocument and put the msgpack binary blob into IR metadata itself.
V2: Removed AMDGPUPALMetadata API used directly by LLPC. The plan now is
that LLPC will continue to put the PAL metadata binary blob into IR metadata itself.
Mar 17 2019
Mar 16 2019
V5: Added vec3 to AMDGPUCallingConvention.td. Fixed call-return-types test for dwordx3 buffer/flat.
(I will add vec5 to AMDGPUCallingConvention.td in the vec5 change.)
V5: Added vec3/vec5 non-shader arg and ret type test cases, including passing by stack.
V4: Properly addressed review comment.
V3: Adjusted the MIR roundtrip fix to use scoping instead of explicit flush.