Подтвердить что ты не робот

Почему оптимизатор не устраняет High в цикле?

Люди сказали, что Delphi производит довольно хороший оптимизированный код для целочисленных операций. Я пробую следующий пример в Delphi 2007 и вижу его код сборки, созданный компилятором.

program p1000;
{$APPTYPE CONSOLE}

procedure test;
var
  arr: array of integer;
  i: integer;

begin
  SetLength(arr, 100);
  for i := 0 to High(arr) do
  begin
    if (i = High(arr)) then
    begin
      arr[i] := -9;
    end;
  end;
end;

begin
  test;
  readln;
end.

Если для конфигурации здания установлено значение DEBUG, я могу установить точку останова и использовать shortkey Ctrl + Alt + D, чтобы увидеть его код сборки, например:

Project3.dpr.11: for i := 0 to High(arr) do
004045A1 8B45FC           mov eax,[ebp-$04]
004045A4 E8F7FAFFFF       call @DynArrayHigh
004045A9 8BF0             mov esi,eax
004045AB 85F6             test esi,esi
004045AD 7C1D             jl $004045cc
004045AF 46               inc esi
004045B0 33DB             xor ebx,ebx
Project3.dpr.13: if (i = High(arr)) then
004045B2 8B45FC           mov eax,[ebp-$04]
004045B5 E8E6FAFFFF       **call @DynArrayHigh**
004045BA 3BD8             cmp ebx,eax
004045BC 750A             jnz $004045c8
Project3.dpr.15: arr[i] := -9;
004045BE 8B45FC           mov eax,[ebp-$04]
004045C1 C70498F7FFFFFF   mov [eax+ebx*4],$fffffff7
Project3.dpr.17: end;
004045C8 43               inc ebx
Project3.dpr.11: for i := 0 to High(arr) do
004045C9 4E               dec esi
004045CA 75E6             jnz $004045b2

Насколько я понимаю, он вызывает функцию High() снова и снова в цикле:

Project3.dpr.13: if (i = High(arr)) then
    004045B2 8B45FC           mov eax,[ebp-$04]
    004045B5 E8E6FAFFFF       **call @DynArrayHigh**
    004045BA 3BD8             cmp ebx,eax

Если для конфигурации здания установлено значение RELEASE, точка останова недоступна, поэтому я нажимаю F8/F7 для перехода в цикл.

00404589 6A64             push $64
0040458B 8D45FC           lea eax,[ebp-$04]
0040458E B901000000       mov ecx,$00000001
00404593 8B1554454000     mov edx,[$00404554]
00404599 E8B6FCFFFF       call @DynArraySetLength
0040459E 83C404           add esp,$04
004045A1 8B45FC           mov eax,[ebp-$04]
004045A4 E8F7FAFFFF       call @DynArrayHigh
004045A9 8BF0             mov esi,eax
004045AB 85F6             test esi,esi
004045AD 7C1D             jl $004045cc
004045AF 46               inc esi
004045B0 33DB             xor ebx,ebx
004045B2 8B45FC           mov eax,[ebp-$04]
004045B5 E8E6FAFFFF       call @DynArrayHigh
004045BA 3BD8             cmp ebx,eax
004045BC 750A             jnz $004045c8
004045BE 8B45FC           mov eax,[ebp-$04]
004045C1 C70498F7FFFFFF   mov [eax+ebx*4],$fffffff7
004045C8 43               inc ebx
004045C9 4E               dec esi
004045CA 75E6             jnz $004045b2
004045CC 33C0             xor eax,eax
004045BC 750A             jnz $004045c8

Снова получается тот же call @DynArrayHigh...
Поэтому мой вопрос: почему компилятор не может его оптимизировать? просто сохраните значение High() в локальном регистре/переменной, потому что размер массива не изменяется.

4b9b3361

Ответ 1

Это не ответ, а комментарий (self destructing):)

На мой взгляд, компилятор не должен пытаться оптимизировать это.

Почему компилятор должен попытаться оптимизировать (не детерминированную) функцию High в отличие от другой? (например, Length)

Длина динамического массива может быть изменена внутри цикла либо на SetLenth, либо другими способами. массив может быть повторно инициализирован во время выполнения, и ваш код может зависеть от этого:

for i := 0 to High(arr) do
begin
  if (i = High(arr)) then
    arr[i] := -9
  else
    if foo() then
      arr := nil; // or SetLength(arr, 0);

  if High(arr) = -1 then Exit; // arr is nil  
end;

Как вы предлагаете, чтобы это было оптимизировано? Должен ли компилятор даже попытаться оптимизировать это? Я не вижу ничего особенного в функции High, даже если компилятор переводит его на @DynArrayHigh.

Если вы хотите, чтобы ваш код был оптимизирован, оптимизируйте его самостоятельно .e.g:

var
  arrHigh: Integer;

  arrHigh := High(arr);
  for i := 0 to arrHigh do
    if i = arrHigh then...