[PATCH] D145146: [Driver][NFC] Remove some redundant code in Driver.cpp.

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 10 12:59:03 PST 2023


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

`computeTargetTriple` has several users. You probably want to check all these code paths do not need `if (const Arg *A = Args.getLastArg(options::OPT_target))`.
For the uses in `BuildCompilation`, it seems fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145146



More information about the cfe-commits mailing list