New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix Issue 9788 - -profile doesn't work if exceptions are thrown in the r... #1806
Conversation
…e running program
@@ -913,27 +913,42 @@ void FuncDeclaration::toObjFile(int multiobj) | |||
bx.module = getModule(); | |||
irs.blx = &bx; | |||
|
|||
/* If profiling, insert call to the profiler here. | |||
* _c_trace_pro(char* funcname); | |||
/* Doing this in semantic3() caused all kinds of problems: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to link this very pull request to illustrate the previous state of affairs: #1806
fix Issue 9788 - -profile doesn't work if exceptions are thrown in the r...
fix Issue 9788 - -profile doesn't work if exceptions are thrown in the r...
se->type = se->type->semantic(0, NULL); | ||
Expressions *exps = new Expressions(); | ||
exps->push(se); | ||
FuncDeclaration *fdpro = FuncDeclaration::genCfunc(Type::tvoid, "trace_pro"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why isn't this _c_trace_pro
any longer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To not break existing binaries. If I broke existing binaries, then using git bisect will fail. I try to keep the binary API the same at least from one release to the next.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I appreciate this and usually try the same, but you missed that goal already because _c_trace_epi is also new.
dlang/druntime@7b162ff
This pull request introduced a regression: |
...unning program
fix http://d.puremagic.com/issues/show_bug.cgi?id=9788