- User Since
- Feb 20 2019, 7:26 AM (121 w, 5 d)
May 13 2021
Maybe we can have a test for this? Otherwise LGTM.
May 7 2021
May 5 2021
Apr 22 2021
Apr 21 2021
@lebedev.ri good to go?
Apr 20 2021
Apr 13 2021
I just want to mention that this is just a starting point, as suggested in D99769. It is definitely not our attention to rush and do this without an RFC or numbers that are in favour of the switch.
I thought we should flip the switch in a separate patch. We would only need to make -enable-attributor=all (or some other configuration) the default.
Apr 12 2021
Apr 8 2021
I'm doing some testing with a lightweight attributor enabled by default. Can this wait few more days, until I get some results? If the numbers are good, maybe we won't need this after all?
Mar 3 2021
I guess the script should be committed first.
Feb 27 2021
Feb 23 2021
Jan 14 2021
Dec 6 2020
Dec 1 2020
Definitely not intentional, sorry.
Nov 24 2020
Nov 23 2020
Nov 21 2020
On the first look, you should at least clang-format this as it is not very readable right now.
Oct 29 2020
Oct 20 2020
Commited in rGfbfb1c790982.
Oct 15 2020
Switch to opt-in approach. Intrinsic class now has the same meaning as bofore.
To get default attributes we now use DefaultAttrsIntrinsic.
Oct 14 2020
Oct 13 2020
Only move the declaration
Oct 12 2020
Oct 2 2020
@nikic do you have a preference on any of the suggested names?
Oct 1 2020
opt-out of few more intrinsics
add comment for DefaultIntrinsic class
update clang tests
Sep 30 2020
Opt-out of default attributes for llvm.coro.* intrinsics.
Actually coro intrinsics had default attributes, completelly missed that. Sorry for the confusion.
I'll update the patch shortly.
Sep 29 2020
precommit (local) coro test changes with update_test_checks
Adding @rjmccall as a reviewer as I'm unsure about some coroutine changes that appeared after rebase.
Its been more than a month now. Are there any additional concern about this?
group default properties together
Sep 28 2020
Sep 22 2020
Switched to D88096.
Now I see where the problem was. Thanks for taking a look.
Sep 19 2020
I think you should split this in two patches. One that introduces the IR attribite and one for attributor deduction.
Sep 9 2020
Sorry for the delay.
Looks like parallel_deletion.ll needs to be updated as well.
Looks same as the failures before. I may be able to do a windows build next week and try to see what is going on.
Sep 7 2020
Actually windows tests passed, I was looking at the wrong place.
I remember this being windows only issue. I was able to always reproduce it on windows, but never had the chance to look into it.
Sep 3 2020
Aug 31 2020
Aug 30 2020
Aug 29 2020
Aug 28 2020
Rebasing after recent changes to AANoUndef
Aug 26 2020
Don't skip ;\n comments
Sorry for the delay on this.