Hi! On Sun, Mar 04, 2018 at 01:52:20PM -0600, Bill Schmidt wrote: > We realized recently that the use of "rs6000" in a builtin name doesn't agree > with our normal naming conventions. Thus this patch changes such a builtin > to __builtin_powerpc_speculation_barrier instead. > > Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is > this okay for trunk?
Most of our builtins have *no* powerpc or similar in the name, and the two that do have "ppc". Should we use __builtin_speculation_barrier? It sounds more likely that other archs will want that as well, than that it will conflict. For the .md patterns we already have various that are named rs6000_* (and none ppc* or power*). Segher