{"head":"6184871242645f1a94ca65609a38a55463f8808d","size":19,"push_id":46273198,"commits":[{"sha":"7e66947f2c849939bbb1346b79302417d8eb121b","author":{"name":"Martin Whitaker","email":"0b9ee6ecb60e38aeb70478538f64d823adb7b95f@martin-whitaker.me.uk"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/7e66947f2c849939bbb1346b79302417d8eb121b","message":"Fix vpiUserDefn definition in vpi_user.h.\n\nThis patch changes the code value for vpiUserDefn to match the IEEE\nstandard."},{"sha":"dc2c67e25fc0183285c26a846bf90d0b638fa82b","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/dc2c67e25fc0183285c26a846bf90d0b638fa82b","message":"Remove two unused argument warnings"},{"sha":"0bad7bc3376c64fd5bda3bcacb31e191c629102b","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/0bad7bc3376c64fd5bda3bcacb31e191c629102b","message":"An enumeration method can be called as a function.\n\nAdd code to allow an enumeration method to be called as a function.\nThis is only the compiler support. The runtime support is still missing\nso only an empty argument call will succeed (e.g. next(), etc.). For now\nthe rest get a warning message."},{"sha":"e22c6926562886fd2edc8a52e2f4b59671738561","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/e22c6926562886fd2edc8a52e2f4b59671738561","message":"Fix a memory leak in ivlpp.\n\nThe new VHDL strings need to be freed to make valgrind happy."},{"sha":"50a2f553647675a7ef54d3057cc09901007da94a","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/50a2f553647675a7ef54d3057cc09901007da94a","message":"Define the mode when testing the width of a wait statement argument.\n\nIf a wait statement has a constant argument then the mode must be defined\nbefore test_width() is called since the current mode is used in the\nPENumber test_width routine."},{"sha":"aa2a063bc06922e0e064dce0b16d1ea6a4c9a7c6","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/aa2a063bc06922e0e064dce0b16d1ea6a4c9a7c6","message":"Remove the use of a freed variable.\n\nIf a constant is padded the original constant will be freed and replaced\nwith a new one. This results in the ce pointer pointing to freed memory.\nIn reality the second check only needs to be done if the first one is\nfalse so avoid the whole issue by only performing the second check if\nthe first on is false."},{"sha":"82e946b51a2dd654e84f08dc22a38a5305c7d45c","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/82e946b51a2dd654e84f08dc22a38a5305c7d45c","message":"Free the iterator if there are extra arguments.\n\nTo avoid extra memory usage on error free the iterator if there are\nextra arguments."},{"sha":"7f65f3b157d6fdc2fffaed2102634e182a8fa3b9","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/7f65f3b157d6fdc2fffaed2102634e182a8fa3b9","message":"Free memory at EOS and convert to C code (vhdl_table.cc).\n\nThis patch converts the vhdl_table.cc file to C code and adds\ncode to free the allocated memory at the end of simulation to\nmake valgrind happy."},{"sha":"5a9e4aaec721ab846457060210d834c8e8e0332b","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/5a9e4aaec721ab846457060210d834c8e8e0332b","message":"When printing a Dec numeric value use the string size and the minimum size.\n\nThe vpiDecStrVal case for the get_numeric() function needs to use the\nexisting string width as the minimum result size. -1 can be represented as\na signed value with a width of 1. This gives a display width of -1 which\nis too small for the -1 string value. This was creating valgrind issues.\nIn general the string value should be the minimum so this is a safe change."},{"sha":"19307ba9dca003f3edcb8ab655dfe362ad9736a8","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/19307ba9dca003f3edcb8ab655dfe362ad9736a8","message":"Add an enum type to the appropriate scope and add valgrind cleanup code.\n\nAdd the enum type to the scope when it is created. Also delete the type\nwhen checking with valgrind."},{"sha":"a3813c006dff5220b22674d86006ffdfbe51b8da","author":{"name":"Martin Whitaker","email":"0b9ee6ecb60e38aeb70478538f64d823adb7b95f@martin-whitaker.me.uk"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/a3813c006dff5220b22674d86006ffdfbe51b8da","message":"Corrected tf_getlongsimtime() to match behaviour of other simulators.\n\nThe PLI tf_getlongsimtime() function is a non-standard function provided\nfor compatibility with some commercial simulators. To match the behaviour\nof those simulators, it should return the raw simulation time, not a\nscaled time."},{"sha":"6a99520fa64af535e19a3357caf9b6112d842eb5","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/6a99520fa64af535e19a3357caf9b6112d842eb5","message":"Add string display for vpiEnumTypespec and create hook for UdpDefn iteration.\n\nThis patch adds vpiEnumTypespec to the list so that vpi_get_str(vpiType, ...)\nreturns an appropriate string when an enum type is passed.\n\nIt also adds the initial hook to allow iteration (global) over the UDP\ndefinitions. This along with table traversal will be needed to free the\nmemory allocated when the UDP definition is created. Ultimately this\nwill all the UDP test to be valgrind clean."},{"sha":"4692756eb9faf2650d14bef0093fb46c860da63e","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/4692756eb9faf2650d14bef0093fb46c860da63e","message":"Add primitive vpiBaseTypespec support for enumerations.\n\nWe don't currently have the information needed to correctly provide the\nbase typespec, but in the next() and prev() methods I need to know if\nthe enumeration is two or four state. This patch sets the base typespec\nfor four state enumerations to vpiReg and to vpiBitVar for two state\nenumerations. This provide enough information to get next() and prev()\nworking correctly."},{"sha":"b905c699d1fb0fa14082ca74fbb11c3e3a6fbe25","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/b905c699d1fb0fa14082ca74fbb11c3e3a6fbe25","message":"Rework enumeration next() and prev() methods to support count argument.\n\nRework the actual next() and prev() methods to correctly process the\nnumeric count argument. Also rework the compiletf routine to give better\nerror messages and combine the call routine for the two methods.\n\nAdd a compiletf routine that checks the arguments and then report that\nthe name() method is not currently implemented."},{"sha":"52d05c021594aa961f7a765cd075a1a872f9d3d9","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/52d05c021594aa961f7a765cd075a1a872f9d3d9","message":"Use correct name to iterate on enumeration constants.\n\nThe standard defines how to calculate the name for an access function\nor iterator if one is not given. It is supposed to be vpi followed by\nthe words in the name with each word capitalized. For the one to many\n(iterator) interface from the enumeration typespec to the individual\nconstants this is vpiEnumConst not vpiMember."},{"sha":"8e3d80679df0eb6333f4ccde123d9eb180a0657d","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/8e3d80679df0eb6333f4ccde123d9eb180a0657d","message":"Add an implementation for the enumeration name() method.\n\nAdd an implementation for the enumeration name() method. This currently\nonly works if the context defines the return width (e.g. the result is\nassigned to a variable). It does not work in a self-determined context\n(e.g. as an argument to a system function or in a comparison). This is\na limitation in the compiler/code generator/run time not the method\nimplementation provided here. We may need full string support to make\nthis work 100%."},{"sha":"8f17e79b7e575536d87021206ff7487aa1a244e7","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/8f17e79b7e575536d87021206ff7487aa1a244e7","message":"Remove a few cppcheck warnings.\n\nThis patch removes a few cppcheck warnings."},{"sha":"209a78a439da68e2d0bdac16bc2580f39cbeb74a","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/209a78a439da68e2d0bdac16bc2580f39cbeb74a","message":"Remove a more cppcheck warnings in vhdlpp.\n\nThis patch removes a few more cppcheck warnings in the vhdlpp directory."},{"sha":"6184871242645f1a94ca65609a38a55463f8808d","author":{"name":"Cary R","email":"44890b136c1e27aad2c22858c9fb74abd9914da6@yahoo.com"},"url":"https://api.github.com/repos/steveicarus/iverilog/commits/6184871242645f1a94ca65609a38a55463f8808d","message":"Remove cppcheck warnings in ivl code.\n\nThis patch removes the new cppcheck warnings in the main (ivl) directory."}],"ref":"refs/heads/master","legacy":{"head":"6184871242645f1a94ca65609a38a55463f8808d","size":19,"push_id":46273198,"shas":[["7e66947f2c849939bbb1346b79302417d8eb121b","mailing-list@martin-whitaker.me.uk","Fix vpiUserDefn definition in vpi_user.h.\n\nThis patch changes the code value for vpiUserDefn to match the IEEE\nstandard.","Martin Whitaker"],["dc2c67e25fc0183285c26a846bf90d0b638fa82b","cygcary@yahoo.com","Remove two unused argument warnings","Cary R"],["0bad7bc3376c64fd5bda3bcacb31e191c629102b","cygcary@yahoo.com","An enumeration method can be called as a function.\n\nAdd code to allow an enumeration method to be called as a function.\nThis is only the compiler support. The runtime support is still missing\nso only an empty argument call will succeed (e.g. next(), etc.). For now\nthe rest get a warning message.","Cary R"],["e22c6926562886fd2edc8a52e2f4b59671738561","cygcary@yahoo.com","Fix a memory leak in ivlpp.\n\nThe new VHDL strings need to be freed to make valgrind happy.","Cary R"],["50a2f553647675a7ef54d3057cc09901007da94a","cygcary@yahoo.com","Define the mode when testing the width of a wait statement argument.\n\nIf a wait statement has a constant argument then the mode must be defined\nbefore test_width() is called since the current mode is used in the\nPENumber test_width routine.","Cary R"],["aa2a063bc06922e0e064dce0b16d1ea6a4c9a7c6","cygcary@yahoo.com","Remove the use of a freed variable.\n\nIf a constant is padded the original constant will be freed and replaced\nwith a new one. This results in the ce pointer pointing to freed memory.\nIn reality the second check only needs to be done if the first one is\nfalse so avoid the whole issue by only performing the second check if\nthe first on is false.","Cary R"],["82e946b51a2dd654e84f08dc22a38a5305c7d45c","cygcary@yahoo.com","Free the iterator if there are extra arguments.\n\nTo avoid extra memory usage on error free the iterator if there are\nextra arguments.","Cary R"],["7f65f3b157d6fdc2fffaed2102634e182a8fa3b9","cygcary@yahoo.com","Free memory at EOS and convert to C code (vhdl_table.cc).\n\nThis patch converts the vhdl_table.cc file to C code and adds\ncode to free the allocated memory at the end of simulation to\nmake valgrind happy.","Cary R"],["5a9e4aaec721ab846457060210d834c8e8e0332b","cygcary@yahoo.com","When printing a Dec numeric value use the string size and the minimum size.\n\nThe vpiDecStrVal case for the get_numeric() function needs to use the\nexisting string width as the minimum result size. -1 can be represented as\na signed value with a width of 1. This gives a display width of -1 which\nis too small for the -1 string value. This was creating valgrind issues.\nIn general the string value should be the minimum so this is a safe change.","Cary R"],["19307ba9dca003f3edcb8ab655dfe362ad9736a8","cygcary@yahoo.com","Add an enum type to the appropriate scope and add valgrind cleanup code.\n\nAdd the enum type to the scope when it is created. Also delete the type\nwhen checking with valgrind.","Cary R"],["a3813c006dff5220b22674d86006ffdfbe51b8da","mailing-list@martin-whitaker.me.uk","Corrected tf_getlongsimtime() to match behaviour of other simulators.\n\nThe PLI tf_getlongsimtime() function is a non-standard function provided\nfor compatibility with some commercial simulators. To match the behaviour\nof those simulators, it should return the raw simulation time, not a\nscaled time.","Martin Whitaker"],["6a99520fa64af535e19a3357caf9b6112d842eb5","cygcary@yahoo.com","Add string display for vpiEnumTypespec and create hook for UdpDefn iteration.\n\nThis patch adds vpiEnumTypespec to the list so that vpi_get_str(vpiType, ...)\nreturns an appropriate string when an enum type is passed.\n\nIt also adds the initial hook to allow iteration (global) over the UDP\ndefinitions. This along with table traversal will be needed to free the\nmemory allocated when the UDP definition is created. Ultimately this\nwill all the UDP test to be valgrind clean.","Cary R"],["4692756eb9faf2650d14bef0093fb46c860da63e","cygcary@yahoo.com","Add primitive vpiBaseTypespec support for enumerations.\n\nWe don't currently have the information needed to correctly provide the\nbase typespec, but in the next() and prev() methods I need to know if\nthe enumeration is two or four state. This patch sets the base typespec\nfor four state enumerations to vpiReg and to vpiBitVar for two state\nenumerations. This provide enough information to get next() and prev()\nworking correctly.","Cary R"],["b905c699d1fb0fa14082ca74fbb11c3e3a6fbe25","cygcary@yahoo.com","Rework enumeration next() and prev() methods to support count argument.\n\nRework the actual next() and prev() methods to correctly process the\nnumeric count argument. Also rework the compiletf routine to give better\nerror messages and combine the call routine for the two methods.\n\nAdd a compiletf routine that checks the arguments and then report that\nthe name() method is not currently implemented.","Cary R"],["52d05c021594aa961f7a765cd075a1a872f9d3d9","cygcary@yahoo.com","Use correct name to iterate on enumeration constants.\n\nThe standard defines how to calculate the name for an access function\nor iterator if one is not given. It is supposed to be vpi followed by\nthe words in the name with each word capitalized. For the one to many\n(iterator) interface from the enumeration typespec to the individual\nconstants this is vpiEnumConst not vpiMember.","Cary R"],["8e3d80679df0eb6333f4ccde123d9eb180a0657d","cygcary@yahoo.com","Add an implementation for the enumeration name() method.\n\nAdd an implementation for the enumeration name() method. This currently\nonly works if the context defines the return width (e.g. the result is\nassigned to a variable). It does not work in a self-determined context\n(e.g. as an argument to a system function or in a comparison). This is\na limitation in the compiler/code generator/run time not the method\nimplementation provided here. We may need full string support to make\nthis work 100%.","Cary R"],["8f17e79b7e575536d87021206ff7487aa1a244e7","cygcary@yahoo.com","Remove a few cppcheck warnings.\n\nThis patch removes a few cppcheck warnings.","Cary R"],["209a78a439da68e2d0bdac16bc2580f39cbeb74a","cygcary@yahoo.com","Remove a more cppcheck warnings in vhdlpp.\n\nThis patch removes a few more cppcheck warnings in the vhdlpp directory.","Cary R"],["6184871242645f1a94ca65609a38a55463f8808d","cygcary@yahoo.com","Remove cppcheck warnings in ivl code.\n\nThis patch removes the new cppcheck warnings in the main (ivl) directory.","Cary R"]],"ref":"refs/heads/master"}}