-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[SandboxIR] Update visibility of IR constructors. #98107
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
Conversation
All SandboxIR objects are owned by sandboxir::Context and should be created using sandboxir::Context's builder functions. This patch protects the constructors to prohibit their use by the user.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/3/builds/1282 Here is the relevant piece of the build log for the reference:
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/72/builds/987 Here is the relevant piece of the build log for the reference:
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/95/builds/1048 Here is the relevant piece of the build log for the reference:
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/176/builds/875 Here is the relevant piece of the build log for the reference:
|
All SandboxIR objects are owned by sandboxir::Context and should be created using sandboxir::Context's builder functions. This patch protects the constructors to prohibit their use by the user.
All SandboxIR objects are owned by sandboxir::Context and should be created using sandboxir::Context's builder functions.
This patch protects the constructors to prohibit their use by the user.