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

LLVM IR pane improvements #5078

Merged
merged 8 commits into from
Jun 11, 2023
Merged

LLVM IR pane improvements #5078

merged 8 commits into from
Jun 11, 2023

Conversation

jeremy-rifkin
Copy link
Member

This PR will add filtering and other output options to the LLVM IR pane

image
image

Closes #5062
Related to #5045

@jeremy-rifkin jeremy-rifkin marked this pull request as draft May 24, 2023 20:16
@github-actions github-actions bot added the ui label May 24, 2023
@OfekShilon
Copy link
Member

Is there anything holding this back from being merged?

@jeremy-rifkin
Copy link
Member Author

jeremy-rifkin commented Jun 7, 2023

Just need to implement the options and filters on the back end, I've been slow getting back to this. Got sidetracked with some other stuff 😄

@jeremy-rifkin jeremy-rifkin marked this pull request as ready for review June 10, 2023 17:47
@jeremy-rifkin
Copy link
Member Author

Should be about there now, cc @OfekShilon.

I'm not sure I'm entirely happy with the filtering choices / behavior yet

@OfekShilon
Copy link
Member

Should be about there now, cc @OfekShilon.

I'm afraid I'm unable to see any difference - not on the live site nor on a local instance. Was this reverted?

@jeremy-rifkin
Copy link
Member Author

It hasn’t been merged into the main branch yet

Copy link
Member Author

@jeremy-rifkin jeremy-rifkin left a comment

Choose a reason for hiding this comment

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

Alright, updated the filtering behavior and now should be good to go

@jeremy-rifkin jeremy-rifkin merged commit 10851a1 into main Jun 11, 2023
13 checks passed
@jeremy-rifkin jeremy-rifkin deleted the jr/ir-pane-improvements branch June 11, 2023 21:00
@partouf
Copy link
Contributor

partouf commented Jun 14, 2023

this is now live

@OfekShilon
Copy link
Member

You guys are the best

@ghost ghost mentioned this pull request Jun 16, 2023
@OfekShilon
Copy link
Member

@jeremy-rifkin Could it be that the symbol demangling option doesn't work yet?

image

@jeremy-rifkin
Copy link
Member Author

Yeah, there's a caching issue (more or less). Fixed it in #5153 but haven't deployed yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REQUEST]: LLVM-IR suggested enhancements
4 participants