Conanap added inline comments.
================ Comment at: clang/include/clang/Basic/BuiltinsPPC.def:50 BUILTIN(__builtin_ppc_compare_and_swaplp, "iLiD*Li*Li", "") +BUILTIN(__builtin_ppc_tdw, "vLLiLLiIi", "") +BUILTIN(__builtin_ppc_tw, "viiIi", "") ---------------- NeHuang wrote: > definition here not matching prototype in document > ``` > void __tdw ( long a, long b, unsigned int TO); > ``` the document had been incorrect; the document is now updated to to `long long`. ================ Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:2233 +def : InstAlias<"tdlle $rA, $rB", (TD 6, g8rc:$rA, g8rc:$rB)>; +def : InstAlias<"tdlge $rA, $rB", (TD 5, g8rc:$rA, g8rc:$rB)>; ---------------- amyk wrote: > Are `tdne`, `tweq` supposed to be aliases that are intended to be added here, > too? the compiler seem to already know `tdne` and `tweq` so I didn't add those. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103668/new/ https://reviews.llvm.org/D103668 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
