[PATCH] D83892: [clang][cli] Port CodeGen option flags to new option parsing system

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 16 08:03:58 PST 2020


probinson added a comment.

One thing this patch does, is make decisions about default behavior static.  Meaning, the option behavior cannot depend on other options; specifically, it can't be based on the triple, which allows target-specific customization.  PS4 certainly has cases where our defaults are different from the usual ones, and I'd kind of think that was true for other targets as well.

Sorry I didn't notice this patch before, our CI has just tried to merge it.  We've patched it up in our main branch but I'm not sure what the upstream intent is here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83892/new/

https://reviews.llvm.org/D83892



More information about the cfe-commits mailing list