@Xiangling_L, sorry if this is adding extra respin test cycles. I've done as much review this time around as I can do for the patch as it is currently posted. Thanks.
Minor comment: Usage of the "ppc" tag seems to imply it is a generic tag for anything Power-related. It makes sense to add it here.
Not sure if other people have a different opinion about this, but I think being explicit for some settings used by "nearby" entries make sense.
Most configurations using involving Ninja appear to use getCmakeWithNinjaBuildFactory. The ones that don't mostly also have make='ninja'.
My understanding is that "llvm" is not needed in the LLVM_ENABLE_PROJECTS list. I guess we could keep it, but I don't think we should keep the trailing semicolon.
Generator options already seem to exist in the ClangBuilder implementation, so I don't think we should be passing them here.
The binutils directory here is used for the gold plugin, so we don't need it for AIX.
Looks like the jobs property on the worker will set the lit threads, so we don't need that part I think.
Just noting we'll have to boost this when we add the native builder for example (we're still waiting on more machine resources though).
LGTM with some last changes based on @daltenty's comments.
The make='ninja' line is not needed given @daltenty's comment.
@daltenty, just confirming you are saying that https://reviews.llvm.org/source/zorg/browse/main/zorg/buildbot/builders/ClangBuilder.py gives us Ninja by default?
Certainly seems that way, every generator option is hard coded to Ninja