Skip to content
This repository has been archived by the owner on Mar 2, 2023. It is now read-only.

Consider giving names to split functions #7

Open
yotann opened this issue Feb 1, 2019 · 2 comments
Open

Consider giving names to split functions #7

yotann opened this issue Feb 1, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@yotann
Copy link
Owner

yotann commented Feb 1, 2019

Because split functions have no name, the globalopt pass (included in opt -O1) deletes them. If we give all the split functions a standardized name (like f) this won't be a problem. However, any name we choose could potentially conflict with other names used by the program.

Another option: store split functions without a name, but give users the option to add a name when retrieving a function from the BCDB.

@yotann yotann added the enhancement New feature or request label Feb 1, 2019
@regehr
Copy link

regehr commented Feb 1, 2019

optionally naming would be excellent for my use case!

@regehr
Copy link

regehr commented Feb 2, 2019

I wrote a silly pass to name these functions, so feel free to deprioritize this if it doesn't help anyone but me

@yotann yotann transferred this issue from yotann/bcdb-private Apr 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants