- User Since
- Jul 29 2016, 12:33 AM (192 w, 4 d)
Sat, Apr 4
Will land this now, because it's a weekend and the previous patch is breaking the buildbots.
iIsEHPadFunctionsSetup -> IsEHPadFunctionsSetUp
Fri, Apr 3
- Change the static variable init to a class member variable IsEHFunctionsSetup (Not sure whether I should AreEHFunctionsSetup instead, but I think usually variable names start with Is..? Let me know if this sounds weird.)
- Move class member variable ExecutedLSDA to an auto variable in prepareEHPads. This variable is used only within that function.
- Add more comment
- Comment fix
There seem to be several other places in which we use Function::Create in this file.. Should we mark all of them as imported as well?
Wed, Apr 1
Yeah then it should be fine to add them in Emscripten side.
(Virtually) talked in person and I have better context for this change now. It'd be better we can add a test for this.
- Is this for static linking or dynamic linking? How did things work before this?
- Is there any tests?
Tue, Mar 31
- Test fix
Thu, Mar 26
- Add comment
Wed, Mar 25
Hmm, come to think of it, I can add a test change that cannot run without this change after all. Thanks for suggestions. Done in D76823.
V8 validation fails without this. I'm not sure if there's a way test v8 validation w/ LLVM unit tests?
Move inline comments to their own lines
Tue, Mar 24
Fri, Mar 20
Thu, Mar 19
Sun, Mar 15
I think the full diff still has not been uploaded yet; everywhere says "Context not available". Could you use arc diff to upload the diff?
Feb 25 2020
Nice, looks a lot simpler!
Feb 22 2020
This part has kind of complicated history and I hope this can be simplified someday 😂... Btw can the test case be simplified? Do we need all that code to generate the pattern?
Feb 21 2020
There seem to be multiple places that call Graph.getLoopEntries(). I guess we should sort all of them..? How about sort LoopEntries at the end of calculate() once and for all?
Feb 14 2020
Yay! 🎉🎉🎉 It's very simpler and nicer this way.
Feb 13 2020
Feb 8 2020
Feb 7 2020
Feb 5 2020
I also did a quick search for what other backends are doing, and they are all kind of different. Many targets seem to implement some function called isEligibleForTailCallOptimization, even though the superclass TargetLowering does not have that method. Anyway, I think it would be fine if we add things we need on an as-needed basis.
Feb 4 2020
I still don't have 100% understanding of what the def issue was, but I can catch up later!
Jan 24 2020
rebase + test fix
Fix test comments
- Remove a stray attribute number
- Test target feature section
Jan 23 2020
I think their implemention is mostly complete, but please let me know if not.
Nice work! Sorry for the delayed reply. I like the test generation script and the autogenerated test case, and given the nature of the tests, I don't think there will be many changes to the generated results. With larger number of limits we can maybe use the script like a fuzzer or something too.
Jan 21 2020
LGTM. Please rebase this onto https://reviews.llvm.org/D71484; this currently contains diff of that CL too.