Siddharth Tewari
Siddharth Tewari
> Here are some issues you can do before we review your PR: > > 1. Some Linux-related CIs failed in the testing process with a segfault. You can reproduce...
> > Most CIs should now be passing. Working on fixing the windows one now. Some spots require strict type conversion. Will check on it. Will push related tests as...
@hydai Can I include the tests within `ExecutorTest.cpp` itself or would it be better to create a new file itself within the Executor folder?
> So, we don't even need the `CoreDump` class. It seems like this class can be replaced by its internal functions directly. There is no member inside the `CoreDump` class....
> I am not available during the weekend, so please make sure you put the most updates before Friday night. That will be my last review for your PR before...
> I don't think so. We will have the Mid-Autumn Festival. And most of us are OOO until 9/18. Ah alright then. I'll push as much as i can by...
- I have added the globals section support. One potential issue is that the segment being inserted to construct the section requires its `Expression` to be set as well and...
> Please note that the GSoC committee will review your PR and implementation status during the final evaluation after the mentor's final evaluation deadline. I will continue to review your...