{"push_id":1426199149,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"dfb3f498f23ac0f29b07d42ecbe0020e5a8df065","before":"974ca2fdbf174adf121158ba12bbf6bcde7e4d29","commits":[{"sha":"f82266fe13f286c5a03b620ffaf21659bad7c053","author":{"name":"Adam Nemet","email":"83a6a0c0be29416f5c01d64e7a687006e7b3bf4b@apple.com"},"message":"[GVN] Basic optimization remark support\n\nFollow-on patches will add more interesting cases.\n\nThe goal of this patch-set is to get the GVN messages printed in\nopt-viewer from Dhrystone as was presented in my Dev Meeting talk.  This\nis the optimization view for the function (the last remark in the\nfunction has a bug which is fixed in this series):\nhttp://lab.llvm.org:8080/artifacts/opt-view_test-suite/build/SingleSource/Benchmarks/Dhrystone/CMakeFiles/dry.dir/html/_org_test-suite_SingleSource_Benchmarks_Dhrystone_dry.c.html#L430\n\nDifferential Revision: https://reviews.llvm.org/D26488\n\ngit-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@288046 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/llvm-beanz/llvm-submodules/commits/f82266fe13f286c5a03b620ffaf21659bad7c053"},{"sha":"dfb3f498f23ac0f29b07d42ecbe0020e5a8df065","author":{"name":"Adam Nemet","email":"83a6a0c0be29416f5c01d64e7a687006e7b3bf4b@apple.com"},"message":"[GVN, OptDiag] Include the value that is forwarded in load elimination\n\nThis requires some changes to the opt-diag API.  Hal and I have\ndiscussed this at the Dev Meeting and came up with a streaming delimiter\n(setExtraArgs) to solve this.\n\nArguments after this delimiter are only included in the optimization\nrecords and not in the remarks printed in the compiler output.  (Note,\nhow in the test the content of the YAML file changes but the remarks on\nthe compiler output don't.)\n\nThis implements the green GVN message with a bug fix at line\nhttp://lab.llvm.org:8080/artifacts/opt-view_test-suite/build/SingleSource/Benchmarks/Dhrystone/CMakeFiles/dry.dir/html/_org_test-suite_SingleSource_Benchmarks_Dhrystone_dry.c.html#L446\n\nThe fix is that now we properly include the constant value in the\nmessage: \"load of type i32 eliminated in favor of 7\"\n\nDifferential Revision: https://reviews.llvm.org/D26489\n\ngit-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@288047 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/llvm-beanz/llvm-submodules/commits/dfb3f498f23ac0f29b07d42ecbe0020e5a8df065"}]}
{"actor":{"display_login":"llvm-beanz"}}