s/ifdef ET_LOG_ENABLED/if ET_LOG_ENABLED/

[swolchok]

Looks like we define it to 0 rather than leaving it undefined to disable.

[swolchok]

Stack from ghstack (oldest at bottom):

• executorch-the-CMake-target should re-export executorch_core, shouldn't it? #8404
• -> s/ifdef ET_LOG_ENABLED/if ET_LOG_ENABLED/ #8403

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/8403

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure

As of commit d5887fc with merge base ee7d388 ():

NEW FAILURE - The following job has failed:

• Check Labels / Check labels (gh)
  RuntimeError: Error checking labels: PR does not have required labels

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[cccclai]

This PR seems reasonable but not sure how it fixes the CI..

[swolchok]

how it fixes the CI.

when ET_LOG_DISABLED was defined to 0 (disabling logging), we were creating the strings to log (gated on #ifdef) but ET_LOG itself was disabled (gated on #if), so the strings were unused, triggering -Wunused-variable.