-
-
Notifications
You must be signed in to change notification settings - Fork 419
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
Remove infinite loop workaround #4068
Conversation
cab3e1d
to
598def3
Compare
The original PR (#2592) also enabled |
If I understand correctly @ergl, that you be to remove this line as the rest is from existing code that was moved around:
correct? |
@SeanTAllen Yes, I would remove that line, the rest was just moving things around, as you mention. |
I agree about no longer needing the "trap unreachable" setting anymore. |
Several years ago, Benoit added code to work around LLVM's lack of support for properly functioning infinite loops. LLVM added support for infinite loops. With this commit, we are removing Benoit's temporary fix.
598def3
to
6ec4b5a
Compare
Several years ago, Benoit added code to work around LLVM's lack
of support for properly functioning infinite loops.
LLVM added support for infinite loops.
With this commit, we are removing Benoit's temporary fix.