From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22473 invoked by alias); 29 Nov 2004 07:41:52 -0000 Mailing-List: contact gcc-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-owner@gcc.gnu.org Received: (qmail 22396 invoked from network); 29 Nov 2004 07:41:47 -0000 Received: from unknown (HELO mtagate2.de.ibm.com) (195.212.29.151) by sourceware.org with SMTP; 29 Nov 2004 07:41:47 -0000 Received: from d12nrmr1607.megacenter.de.ibm.com (d12nrmr1607.megacenter.de.ibm.com [9.149.167.49]) by mtagate2.de.ibm.com (8.12.10/8.12.10) with ESMTP id iAT7fkbP153126 for ; Mon, 29 Nov 2004 07:41:46 GMT Received: from d12av02.megacenter.de.ibm.com (d12av02.megacenter.de.ibm.com [9.149.165.228]) by d12nrmr1607.megacenter.de.ibm.com (8.12.10/NCO/VER6.6) with ESMTP id iAT7ftiG126206 for ; Mon, 29 Nov 2004 08:41:55 +0100 Received: from d12av02.megacenter.de.ibm.com (loopback [127.0.0.1]) by d12av02.megacenter.de.ibm.com (8.12.11/8.12.11) with ESMTP id iAT7fkOY030403 for ; Mon, 29 Nov 2004 08:41:46 +0100 Received: from d12ml102.megacenter.de.ibm.com (d12ml102.megacenter.de.ibm.com [9.149.166.138]) by d12av02.megacenter.de.ibm.com (8.12.11/8.12.11) with ESMTP id iAT7fjAa030400; Mon, 29 Nov 2004 08:41:45 +0100 In-Reply-To: Subject: Re: SMS parameters To: Nathan Sidwell Cc: gcc Message-ID: From: Mostafa Hagog Date: Mon, 29 Nov 2004 09:59:00 -0000 MIME-Version: 1.0 Content-type: multipart/mixed; Boundary="0__=4DBBE5C8DFBBB6D48f9e8a93df938690918c4DBBE5C8DFBBB6D4" Content-Disposition: inline X-SW-Source: 2004-11/txt/msg01111.txt.bz2 --0__=4DBBE5C8DFBBB6D48f9e8a93df938690918c4DBBE5C8DFBBB6D4 Content-type: text/plain; charset=US-ASCII Content-length: 1266 Nathan Sidwell wrote on 26/11/2004 13:02:23: > The following sms parameters exist. > > max-sms-loop-number > sms-max-ii-factor > sms-dfa-history > sms-loop-average-count-threshold > > None of these are documented in invoke.texi. I have attached a patch for this. > > The first one is odd, as its help string says, > "Maximum number of loops to perform swing modulo scheduling on \ > (mainly for debugging)" > and its implementation is via a static counter that counts bbs in a *program* > > static int passes = 0; > ... > FOR_EACH_BB (bb) > ... > /* For debugging. */ > if ((passes++ > MAX_SMS_LOOP_NUMBER) && (MAX_SMS_LOOP_NUMBER != -1)) > > it doesn't look like a particularly sensible thing to expose. You are correct, this is used for debugging usually by gcc developers, I use it intensively in debugging and tuning SMS. How can we hide this parameter from the GCC users and keep it for the developers? In the meanwhile I didn't add documentation for this parameter to invoke.texi > If it must be exposed, it should probably be called 'sms-max-loop-number'. > Notice the fence-post error there, we'll actually schedule 1 more > than MAX_SMS_LOOP_NUMBER. Agree. Mostafa. (See attached file: sms_params_doc.patch) --0__=4DBBE5C8DFBBB6D48f9e8a93df938690918c4DBBE5C8DFBBB6D4 Content-type: application/octet-stream; name="sms_params_doc.patch" Content-Disposition: attachment; filename="sms_params_doc.patch" Content-transfer-encoding: base64 Content-length: 2485 SW5kZXg6IHBhcmFtcy5kZWYKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1Mg ZmlsZTogL2N2cy9nY2MvZ2NjL2djYy9wYXJhbXMuZGVmLHYKcmV0cmlldmlu ZyByZXZpc2lvbiAxLjQ4CmRpZmYgLWMgLXAgLXIxLjQ4IHBhcmFtcy5kZWYK KioqIHBhcmFtcy5kZWYJMjcgT2N0IDIwMDQgMjE6MDI6MDcgLTAwMDAJMS40 OAotLS0gcGFyYW1zLmRlZgkyOSBOb3YgMjAwNCAwNzozNzoxNyAtMDAwMAoq KioqKioqKioqKioqKiogREVGUEFSQU0oUEFSQU1fTUFYX0lURVJBVElPTlNf VE9fVFJBQ0ssCioqKiAyNjYsMjcyICoqKioKICAJMTAwMCkKICAKICBERUZQ QVJBTShQQVJBTV9NQVhfU01TX0xPT1BfTlVNQkVSLAohIAkgIm1heC1zbXMt bG9vcC1udW1iZXIiLAogIAkgIk1heGltdW0gbnVtYmVyIG9mIGxvb3BzIHRv IHBlcmZvcm0gc3dpbmcgbW9kdWxvIHNjaGVkdWxpbmcgb24gXAogIAkgICht YWlubHkgZm9yIGRlYnVnZ2luZykiLAogIAkgLTEpCi0tLSAyNjYsMjcyIC0t LS0KICAJMTAwMCkKICAKICBERUZQQVJBTShQQVJBTV9NQVhfU01TX0xPT1Bf TlVNQkVSLAohIAkgInNtcy1tYXgtbG9vcC1udW1iZXIiLAogIAkgIk1heGlt dW0gbnVtYmVyIG9mIGxvb3BzIHRvIHBlcmZvcm0gc3dpbmcgbW9kdWxvIHNj aGVkdWxpbmcgb24gXAogIAkgIChtYWlubHkgZm9yIGRlYnVnZ2luZykiLAog IAkgLTEpCkluZGV4OiBkb2MvaW52b2tlLnRleGkKPT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PQpSQ1MgZmlsZTogL2N2cy9nY2MvZ2NjL2djYy9kb2MvaW52b2tl LnRleGksdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuNTU4CmRpZmYgLWMgLXAg LXIxLjU1OCBpbnZva2UudGV4aQoqKiogZG9jL2ludm9rZS50ZXhpCTIzIE5v diAyMDA0IDE3OjM5OjU1IC0wMDAwCTEuNTU4Ci0tLSBkb2MvaW52b2tlLnRl eGkJMjkgTm92IDIwMDQgMDc6Mzc6MjUgLTAwMDAKKioqKioqKioqKioqKioq IGluZHVjdGlvbiB2YXJpYWJsZSB1c2VzLgoqKiogNTQ4Myw1NDg4ICoqKioK LS0tIDU0ODMsNTUwMyAtLS0tCiAgVGhlIG1heGltdW0gbnVtYmVyIG9mIGl0 ZXJhdGlvbnMgb2YgYSBsb29wIHRoZSBicnV0ZSBmb3JjZSBhbGdvcml0aG0K ICBmb3IgYW5hbHlzaXMgb2YgIyBvZiBpdGVyYXRpb25zIG9mIHRoZSBsb29w IHRyaWVzIHRvIGV2YWx1YXRlLgogIAorIEBpdGVtIHNtcy1tYXgtaWktZmFj dG9yCisgCisgQSBmYWN0b3IgZm9yIHR1bmluZyB0aGUgdXBwZXIgYm91bmQg b2YgdGhlIEluaXRpYXRpb24gSW50ZXJ2YWwgdGhhdCAKKyBzd2luZyBtb2R1 bG8gc2NoZWR1bGVyIHVzZXMgZm9yIHNjaGVkdWxpbmcgYSBsb29wLgorIAor IEBpdGVtIHNtcy1kZmEtaGlzdG9yeQorIAorIFRoZSBudW1iZXIgb2YgY3lj bGVzIHRoZSBzd2luZyBtb2R1bG8gc2NoZWR1bGVyIGNvbnNpZGVycyB3aGVu IGNoZWNraW5nIAorIGNvbmZsaWN0cyB1c2luZyBERkEuCisgCisgQGl0ZW0g c21zLWxvb3AtYXZlcmFnZS1jb3VudC10aHJlc2hvbGQKKyAKKyBBIHRocmVz aG9sZCBvbiB0aGUgYXZlcmFnZSBjb3VudCBmb3IgbG9vcHMgdG8gYmUgY29u c2lkZXJlZCBieSB0aGUgc3dpbmcgCisgbW9kdWxvIHNjaGVkdWxlci4KKyAK ICBAaXRlbSBob3QtYmItY291bnQtZnJhY3Rpb24KICBTZWxlY3QgZnJhY3Rp b24gb2YgdGhlIG1heGltYWwgY291bnQgb2YgcmVwZXRpdGlvbnMgb2YgYmFz aWMgYmxvY2sgaW4gcHJvZ3JhbQogIGdpdmVuIGJhc2ljIGJsb2NrIG5lZWRz IHRvIGhhdmUgdG8gYmUgY29uc2lkZXJlZCBob3QuCg== --0__=4DBBE5C8DFBBB6D48f9e8a93df938690918c4DBBE5C8DFBBB6D4--