LuoYuanke added inline comments.
================ Comment at: clang/include/clang/Driver/Options.td:4651 def mno_popcnt : Flag<["-"], "mno-popcnt">, Group<m_x86_Features_Group>; +def mprefetchi : Flag<["-"], "mprefetchi">, Group<m_x86_Features_Group>; +def mno_prefetchi : Flag<["-"], "mno-prefetchi">, Group<m_x86_Features_Group>; ---------------- I notice in line 4655 the option name is "prfch", do we need to follow the naming convention? ================ Comment at: llvm/lib/Target/X86/X86InstrInfo.td:3007 + def PREFETCHIT0 : I<0x18, MRM7m, (outs), (ins i8mem:$src), + "prefetchit0\t$src", [(prefetch addr:$src, (i32 1), (i32 3), (i32 0))]>, TB; + def PREFETCHIT1 : I<0x18, MRM6m, (outs), (ins i8mem:$src), ---------------- Could you add comments to explain the what the constant (1, 3, 0) means? I guess it is the same to the arguments that llvm.prefetch defines. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136040/new/ https://reviews.llvm.org/D136040 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits