{"action":"created","issue":{"url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/120","labels_url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/120/labels{/name}","comments_url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/120/comments","events_url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/120/events","html_url":"https://github.com/ucsd-progsys/liquid-fixpoint/issues/120","id":104938292,"number":120,"title":"Previously passing tests now fail with Unsafe","user":{"login":"bmcfluff","id":4876098,"avatar_url":"https://avatars.githubusercontent.com/u/4876098?v=3","gravatar_id":"","url":"https://api.github.com/users/bmcfluff","html_url":"https://github.com/bmcfluff","followers_url":"https://api.github.com/users/bmcfluff/followers","following_url":"https://api.github.com/users/bmcfluff/following{/other_user}","gists_url":"https://api.github.com/users/bmcfluff/gists{/gist_id}","starred_url":"https://api.github.com/users/bmcfluff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmcfluff/subscriptions","organizations_url":"https://api.github.com/users/bmcfluff/orgs","repos_url":"https://api.github.com/users/bmcfluff/repos","events_url":"https://api.github.com/users/bmcfluff/events{/privacy}","received_events_url":"https://api.github.com/users/bmcfluff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-09-04T17:56:48Z","updated_at":"2015-09-04T18:26:35Z","closed_at":null,"body":"For example:\r\n\r\nhttps://github.com/ucsd-progsys/liquid-fixpoint/blob/master/external/fixpoint/tests/liquidhaskell_tests/pos/TopLevel.hs.fq\r\n\r\nAnd while I don't have a small test case yet, I think I know why: look at lines 175 and 188 of that file; under the new scheme where binders that match a constant get dropped, we lose the refinement on bind 5 which may have been necessary to prove things Safe."},"comment":{"url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/comments/137812707","html_url":"https://github.com/ucsd-progsys/liquid-fixpoint/issues/120#issuecomment-137812707","issue_url":"https://api.github.com/repos/ucsd-progsys/liquid-fixpoint/issues/120","id":137812707,"user":{"login":"ranjitjhala","id":1650232,"avatar_url":"https://avatars.githubusercontent.com/u/1650232?v=3","gravatar_id":"","url":"https://api.github.com/users/ranjitjhala","html_url":"https://github.com/ranjitjhala","followers_url":"https://api.github.com/users/ranjitjhala/followers","following_url":"https://api.github.com/users/ranjitjhala/following{/other_user}","gists_url":"https://api.github.com/users/ranjitjhala/gists{/gist_id}","starred_url":"https://api.github.com/users/ranjitjhala/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ranjitjhala/subscriptions","organizations_url":"https://api.github.com/users/ranjitjhala/orgs","repos_url":"https://api.github.com/users/ranjitjhala/repos","events_url":"https://api.github.com/users/ranjitjhala/events{/privacy}","received_events_url":"https://api.github.com/users/ranjitjhala/received_events","type":"User","site_admin":false},"created_at":"2015-09-04T18:26:34Z","updated_at":"2015-09-04T18:26:34Z","body":"Seems to me that:\n\nIF\n\n  + you have a binder and constant with the same name `x`,\n\nTHEN\n\n  + all the binders should have the same sort (of course)\n  + all the binders should be free of any KVARs (only CONCRETE refinements)\n\nIn short we NEED NOT rename those binders, because there are no KVARs.\n\nSO: can you modify uniqify (aka renameAll, so that it DOES NOT rename any\nbinders\nthat correspond to constants.)\n\nThanks!\n\nRJ.\n\n\nOn Fri, Sep 4, 2015 at 11:20 AM, Ranjit Jhala <jhala@cs.ucsd.edu> wrote:\n\n> i get it, `True` is the constant/binder in question.\n>\n> Hmm, let me think about this.\n> ​\n>\n\n\n\n-- \nRanjit.\n"}}