User Details
- User Since
- Jul 8 2012, 1:04 PM (559 w, 5 h)
- Roles
- Administrator
Thu, Mar 23
Feb 24 2023
Feb 9 2023
Feb 7 2023
Feb 6 2023
Feb 3 2023
LGTM provided there will be a Discourse link :)
Feb 1 2023
Jan 31 2023
Jan 24 2023
Jul 26 2022
See comments above
Jul 12 2022
Looks like there are some typos in runner names
Jun 11 2022
It seems reasonable to me. Even if it is only used on a single platform, it is small enough from maintenance perspective to have it.
May 23 2022
Thanks!
May 10 2022
Mar 19 2022
@nlopes I think we need to better frame the requirement then, ask for CVs (this is always done though) and maybe even organize some kind of interviews? Strongly favouring ones with prior contributions is a good thing IMO
Mar 14 2022
Feb 24 2022
Jan 27 2022
Neat! :)
Jan 21 2022
Jan 19 2022
Jan 10 2022
LGTM, thanks!
Jan 9 2022
Dec 29 2021
Will you please check the comments and reword everything using proper terminology and new things we're having on GitHub?
Dec 28 2021
Thanks for doing this! I added few notes on the way.
Dec 16 2021
Dec 14 2021
Dec 13 2021
@tstellar Does it work for our "complex" labels? E.g. "lld:ELF" or "tools:llvm-ar/llvm-ranlib"?
Dec 12 2021
Dec 11 2021
Dec 1 2021
Nov 22 2021
We can also put into .github/ISSUE_TEMPLATE/config.yml the following:
blank_issues_enabled: false
To disallow non-templated issues
Oct 25 2021
@thopre I doubt the refactorings of this kind (change one plotting framework into another) could be split into smaller patches just by the nature of the change, you cannot have "half plot" and one essentially needs all the features ready for the review. So, such large patches are inevitable
Sep 29 2021
Indeed, this was an oversight. Bi could be generated from an indirect branch as a result of folding or during the branch relaxation.
Sep 28 2021
Will it be possible to split this patch into 3:
Please see comments inline
LGTM, thanks!
Sep 27 2021
Sep 20 2021
Sorry, I was AFK for a moment. Will review today
Aug 16 2021
In general, this looks reasonable refactoring to me. @jhenderson ?
LGTM, thanks
Mar 18 2021
Indeed! And even worse, there is another 'A' shadowing definition below :)
Mar 17 2021
LGTM, thanks!
Looks trivial enough for post-commit review :)
Feb 4 2021
It seems you pushed to master, not main.
Jan 23 2021
Jan 20 2021
Fixed in cce2c1e
Jan 18 2021
Oct 9 2020
Can we fold all these LLVMBuild.cmake files into the corresponding cmakelists.txt?
Aug 9 2020
Aug 7 2020
Will you please switch to iterators here?
Jul 8 2020
LGTM, thanks!
Jun 30 2020
Jun 26 2020
Jun 25 2020
Jun 22 2020
LGTM, thanks!
Looks good to me! FP was an old register name from mspgcc time. MSP430 EABI does not use frame pointer at all (seems to be a bug in the ABI, as I do not understand how we could support VLAs or alloca's in this case).