Skip to content

Commit 08d920c

Browse files
committed
Auto merge of #42650 - nrc:save-slim, r=eddyb
save-analysis: remove a lot of stuff This commits us to the JSON format and the more general def/ref style of output, rather than also supporting different data formats for different data structures. This does not affect the RLS at all, but will break any clients of the CSV form - AFAIK there are none (beyond a few of my own toy projects) - DXR stopped working long ago. r? @eddyb
2 parents 3438c0f + 8a2857e commit 08d920c

File tree

12 files changed

+844
-3280
lines changed

12 files changed

+844
-3280
lines changed

src/librustc/session/config.rs

-4
Original file line numberDiff line numberDiff line change
@@ -935,8 +935,6 @@ options! {DebuggingOptions, DebuggingSetter, basic_debugging_options,
935935
save_analysis: bool = (false, parse_bool, [UNTRACKED],
936936
"write syntax and type analysis (in JSON format) information, in \
937937
addition to normal output"),
938-
save_analysis_csv: bool = (false, parse_bool, [UNTRACKED],
939-
"write syntax and type analysis (in CSV format) information, in addition to normal output"),
940938
save_analysis_api: bool = (false, parse_bool, [UNTRACKED],
941939
"write syntax and type analysis information for opaque libraries (in JSON format), \
942940
in addition to normal output"),
@@ -2471,8 +2469,6 @@ mod tests {
24712469
assert_eq!(reference.dep_tracking_hash(), opts.dep_tracking_hash());
24722470
opts.debugging_opts.save_analysis = true;
24732471
assert_eq!(reference.dep_tracking_hash(), opts.dep_tracking_hash());
2474-
opts.debugging_opts.save_analysis_csv = true;
2475-
assert_eq!(reference.dep_tracking_hash(), opts.dep_tracking_hash());
24762472
opts.debugging_opts.save_analysis_api = true;
24772473
assert_eq!(reference.dep_tracking_hash(), opts.dep_tracking_hash());
24782474
opts.debugging_opts.print_move_fragments = true;

src/librustc_driver/lib.rs

-3
Original file line numberDiff line numberDiff line change
@@ -534,15 +534,12 @@ impl<'a> CompilerCalls<'a> for RustcDefaultCalls {
534534

535535
fn save_analysis(sess: &Session) -> bool {
536536
sess.opts.debugging_opts.save_analysis ||
537-
sess.opts.debugging_opts.save_analysis_csv ||
538537
sess.opts.debugging_opts.save_analysis_api
539538
}
540539

541540
fn save_analysis_format(sess: &Session) -> save::Format {
542541
if sess.opts.debugging_opts.save_analysis {
543542
save::Format::Json
544-
} else if sess.opts.debugging_opts.save_analysis_csv {
545-
save::Format::Csv
546543
} else if sess.opts.debugging_opts.save_analysis_api {
547544
save::Format::JsonApi
548545
} else {

0 commit comments

Comments
 (0)