public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "hbucher at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug middle-end/109057] Does GCC interpret assembly when deciding to optimize away a variable?
Date: Tue, 07 Mar 2023 17:17:23 +0000	[thread overview]
Message-ID: <bug-109057-4-7MWsmtX5sr@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-109057-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109057

--- Comment #1 from Henry <hbucher at gmail dot com> ---
Two caveats: 

1. If you add something like `xor %0,%0` inside the assembly text, LUT is not
optimized

inline void DoNotOptimize( uint8_t value) {
  asm volatile("xor %0,%0" : : "r,m"(value) : "memory");
}
void func2(uint8_t val) {
    DoNotOptimize(LUT[val]); 
}

Produces

func2(unsigned char):
        movzbl  %dil, %edi
        xor LUT(%rdi),LUT(%rdi)
        ret
LUT:
        .string "\001\005\003"
        .ascii  "\002\007\001\002"

https://godbolt.org/z/Mn5asGWe4

2. If you make value a uint32_t instead of a uint8_t, LUT is not optimized

#include <stdint.h>
static const uint8_t LUT[8] = {1,5,3,0,2,7,1,2};

inline void DoNotOptimize( uint32_t value) {
  asm volatile("" : : "r,m"(value) : "memory");
}
void func2(uint8_t val) {
    DoNotOptimize(LUT[val]); 
}

Produces 

func2(unsigned char):
        movzbl  %dil, %edi
        movzbl  LUT(%rdi), %eax
        ret
LUT:
        .string "\001\005\003"
        .ascii  "\002\007\001\002"

https://godbolt.org/z/rTfExvEbb

  reply	other threads:[~2023-03-07 17:17 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-07 17:07 [Bug c++/109057] New: " hbucher at gmail dot com
2023-03-07 17:17 ` hbucher at gmail dot com [this message]
2023-03-07 17:22 ` [Bug middle-end/109057] " pinskia at gcc dot gnu.org
2023-03-07 17:27 ` jakub at gcc dot gnu.org
2023-03-07 17:28 ` hbucher at gmail dot com
2023-03-07 17:29 ` jakub at gcc dot gnu.org
2023-03-07 17:31 ` hbucher at gmail dot com
2023-03-07 17:32 ` jakub at gcc dot gnu.org
2023-03-07 17:32 ` pinskia at gcc dot gnu.org
2023-03-07 17:38 ` hbucher at gmail dot com
2023-03-07 17:44 ` jakub at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-109057-4-7MWsmtX5sr@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).