User Details
- User Since
- Feb 20 2019, 7:26 AM (99 w, 3 d)
Thu, Jan 14
Dec 6 2020
Dec 1 2020
Definitely not intentional, sorry.
Nov 24 2020
Cool :)
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.
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.
LGTM
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
LGTM, thanks!
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.
Are you going to move the AAIsDead query to the update?
Thanks for the explanation. Since the patch is not small and it is a WIP, can you add some more information about the failing tests? Maybe some specific examples?
Aug 25 2020
Aug 24 2020
What should be the next step here? There was no response on the RFC update.
Does the fix for the performance issue seem ok?
Aug 22 2020
LGTM, thanks!
Aug 21 2020
LGTM.
We can't delete functions outside of the SCC, the rest should be fine.
Aug 20 2020
Thanks for reporting this.
Numbers are similar in the release mode as well. The problem was that we go through all IntrProperties for every Intrinsic in search for ones that are default. However that can be done only once.
fix slowdown issue
Aug 19 2020
Aug 18 2020
Maybe the commit message could be a bit more descriptive?
Aug 17 2020
addressing comments
addressing comments
update affected tests.
add comment for DisableDefaultAttributes flag.
Ping.
Can we move forward with this approach?
update gc intrinsics to DefaultIntrinsic