Skip to content
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

Set error message for where to report bugs #11602

Merged
merged 3 commits into from
Dec 19, 2022
Merged

Conversation

jpienaar
Copy link
Member

@jpienaar jpienaar commented Dec 19, 2022

Avoids recommending filing bugs with LLVM project for IREE project tooling.

@benvanik
Copy link
Collaborator

uh, that's a lot of commits? 😮

@benvanik
Copy link
Collaborator

(otherwise LGTM - just need to clean up whatever happened in git - thanks for doing this!)

@jpienaar
Copy link
Member Author

uh, that's a lot of commits? open_mouth

I don't know why I keep doing this ... I git pull main && git branch && git checkout && git commit && gh pr create ... I should just explicitly have a git fetch gh && git rebase gh/main in there too.

Avoids recommending filing bugs with LLVM project for IREE project
tooling.
@benvanik
Copy link
Collaborator

if you pull main while on main then it should rebase for you - maybe you're issuing your git pull main while on another branch?
my flow is usually:

git checkout main && git pull --prune
git checkout -b new-branch
<work>

@jpienaar jpienaar enabled auto-merge (squash) December 19, 2022 17:49

int main(int argc, char **argv) { return ireeOptRunMain(argc, argv); }
int main(int argc, char **argv) {
llvm::setBugReportMsg(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this calls the method in IREEOptToolEntryPoint.cpp so it shouldn't be needed here

@jpienaar jpienaar merged commit 17ec7ab into iree-org:main Dec 19, 2022
@jpienaar jpienaar deleted the error branch December 19, 2022 21:05
pzread pushed a commit to pzread/iree that referenced this pull request Jan 4, 2023
pzread pushed a commit to pzread/iree that referenced this pull request Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants