- User Since
- Mar 11 2015, 2:39 PM (232 w, 2 d)
Thu, Aug 8
In that case LGTM
Other than my small suggestion about the use of NoRegister, this looks good to me.
Mon, Aug 5
Thu, Aug 1
Rebase on top of tree.
Wed, Jul 31
Mon, Jul 29
Jul 25 2019
Jul 15 2019
As an aside, I've noticed a codegen issue when using floating point clobber lists, resulting in the implicit-defs not being added to INLINEASM instructions. I'm working on a fix for that now and will submit a second patch shortly.
Jul 3 2019
Jun 18 2019
- Refactored register tables to match style used in i386/x86_64
- Add enum for RISC-V DWARF numbers
- Add F registers (assuming 32-bit, at runtime this seems to be overwritten to 64-bit if D extension is provided)
- Add default unwind plan for first frame
May 31 2019
Jan 31 2019
As this mllvm option only affects the creation of ELF objects, do we also need to add a similar option for the LTO case, as the -G value would have no effect otherwise?
Nov 7 2018
Oct 15 2018
I tried building this on top of trunk but my build failed with this assertion:
Aug 15 2018
My tests now look better, there are a couple of failures, but this seems to be a bug in newlib, rather than with clang/this patch (the bug was masked before as we would have been pulling in system headers). So this patch looks good to me.
Aug 14 2018
I've tested this, regression tests involving linking now mostly pass as crt0 can now be found, but it seems that system headers are still being pulled in causing a couple of tests to fail. In particular using limits.h is causing build failures for me.
Aug 2 2018
It seems the ability to link objects has been broken by this change. As an example from our nightly tests:
Jun 21 2018
I think this should also cover mismatched arguments if the attribute appears several times, and reject/warn about the attribute combination in these cases.
May 23 2018
May 22 2018
Note: I've marked this as WIP whilst discussing the interface, I'm not suggesting hardcoding in a setSTI() function to MCAsmBackend, that will be removed from the final patch
Rebased on top of tree, and applied changes as per Alex's review.
I agree, the route you have taken in D46965 seems the best route to go down. Unless we end up with some fixup where we want to manipulate bytes that wouldn't be covered by that calculation (it seems unlikely), then all this table is doing is duplicating what information we already have, so I think it's best to use your calculation.
May 11 2018
May 4 2018
May 3 2018
Apr 19 2018
I've readded the reloc check line to hilo-constaddr.s and renamed hilo-constaddr-invalid.s to hilo-constaddr-expr.s since it is no longer checking something that is always invalid.
Apr 16 2018
I've rebased the change on top of D44886 to indicate what is conditional based on linker relaxation.
Apr 7 2018
Take part of test I removed as it now produces and error, and turn it into an explicit test for an error message.
Apr 6 2018
The previous diff had one change missing from my original patch, that is now restored.
I've updated the patch to respond to comments, and deal with the couple of test failures I was seeing.
Apr 4 2018
Apr 2 2018
Nov 15 2017
Thanks, I found one nitpick with a couple of the tests, but the rest looks good to me.
Nov 14 2017
Nov 1 2016
Aug 24 2016
Aug 17 2016
Jul 19 2016
Jan 26 2016
Sep 22 2015
Added comment to test as per recommendation.