- User Since
- Feb 9 2017, 1:53 PM (87 w, 5 d)
Thu, Oct 11
Wed, Oct 10
Tue, Oct 9
Do you have more information about the change in Ubuntu that caused this to stop working?
Mon, Oct 8
Fri, Oct 5
Here is an AMDGPU patch that uses legalizeIntrinsic() https://reviews.llvm.org/D52923
Thu, Oct 4
Wed, Oct 3
I've implemented an alternative solution here that I think is a little more robust: https://reviews.llvm.org/D52861
Fri, Sep 28
Thu, Sep 27
I've created the 7.0.1 metabug, updated the merge-request.sh script. and created a merge request for this: https://llvm.org/PR39107
Use cmake's DEPRECATION message.
Wed, Sep 26
Tue, Sep 25
Sep 12 2018
Is this more what you had in mind?
Sep 11 2018
Sep 7 2018
Updated. If you have any more feedback you can leave it on the review for the other patch.
Add custom install targets for modules.
I've just updated the patch with some fixes, can you try again?
Fix cmake failure with -DLLVM_LINK_LLVM_DYLIB=OFF
What CMake arguments were you using?
D51748 proposes refactoring this whole function and effectively makes the same change. Would you be able to test this to see if it helps your uses case?
Sep 6 2018
Ok, my mistake.
There are a few other modules in the tree that are unnecessarily installed. So I made an attempt to fix this in a more generic way: D51748.
Sep 5 2018
Rebase on trunk.
Sep 4 2018
My mistake. I had been working on the release_70 branch and did not realize the REQUIRES had been added to trunk when I rebased. Looks like this was fixed in r339253. I'll ask Hans to backport that to the stable branch.
Aug 31 2018
Aug 30 2018
Aug 28 2018
Aug 21 2018
Aug 20 2018
Aug 17 2018
@atrick does this look OK to you?
Aug 16 2018
I had to revert this, because it broke the Windows build. I am working on a new patch.
Aug 14 2018
Use add_llvm_library() instead of add_llvm_loadable_module()
Aug 13 2018
Aug 10 2018
OK, that's good to know, I will drop this patch from Fedora.
Aug 9 2018
Aug 8 2018
Aug 2 2018
Looks good to me.
Aug 1 2018
Jul 31 2018
One questions I had after looking into this bug was why does PPCTargetLowering::LowerFP_TO_INTForReuse() extend f32 values to f64 before creating the FCT* instructions?